Differential D2345

VerifyECDSA
ClosedPublic
Actions

Authored by markblundeberg on Jan 19 2019, 17:13.

Details

Reviewers

deadalnix

Group Reviewers

Restricted Project

Maniphest Tasks

T527: Add Schnorr support to OP_CHECKSIG and OP_CHECKDATASIG

Commits

rSTAGINGec83c14d4a84: API rename: Sign->SignECDSA, Verify->VerifyECDSA
rABCec83c14d4a84: API rename: Sign->SignECDSA, Verify->VerifyECDSA

Summary

This is being done simply for additional clarity when Schnorr sigs are
introduced in future.

Test Plan

Trivial renames -- it compiles = success.

Diff Detail

Repository

rABC Bitcoin ABC

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

markblundeberg created this revision.Jan 19 2019, 17:13

Herald added a reviewer: Restricted Project. · View Herald TranscriptJan 19 2019, 17:13

Herald added a reviewer: deadalnix. · View Herald Transcript

Herald added a subscriber: schancel. · View Herald Transcript

Only one doubt: are there any external programs that call into this API?

Build Bitcoin-ABC / Diffs / Bitcoin-ABC Diff Testing started.

Harbormaster completed remote builds in B4583: Diff 6748.Jan 19 2019, 17:38

Build Bitcoin-ABC / Diffs / Bitcoin-ABC Diff Testing passed.

markblundeberg mentioned this in D2347: API renames of script signature functions to mention ECDSA.Jan 19 2019, 19:37

markblundeberg added a child revision: D2348: Add CKey::SignSchnorr and CPubKey::VerifySchnorr.Jan 19 2019, 20:10

markblundeberg mentioned this in D2342: WIP: add Schnorr opcode support.Jan 19 2019, 21:31

deadalnix accepted this revision.Jan 20 2019, 00:26

This revision is now accepted and ready to land.Jan 20 2019, 00:26

Closed by commit rABCec83c14d4a84: API rename: Sign->SignECDSA, Verify->VerifyECDSA (authored by Mark Lundeberg). · Explain WhyJan 20 2019, 03:20

This revision was automatically updated to reflect the committed changes.

markblundeberg added a task: T527: Add Schnorr support to OP_CHECKSIG and OP_CHECKDATASIG.Jan 20 2019, 07:31

Revision Contents
Changeset List

Path

Size

src/

13 lines

8 lines

11 lines

4 lines

script/

interpreter.cpp

2 lines

sign.cpp

2 lines

test/

checkdatasig_tests.cpp

2 lines

key_tests.cpp

56 lines

multisig_tests.cpp

2 lines

script_tests.cpp

10 lines

txvalidationcache_tests.cpp

10 lines

Diff 6757

View Options

src/key.h

View Options

src/key.cpp

View Options

src/pubkey.h

View Options

src/pubkey.cpp

View Options

src/script/interpreter.cpp

View Options

src/script/sign.cpp

View Options

src/test/checkdatasig_tests.cpp

View Options

src/test/key_tests.cpp

View Options

src/test/multisig_tests.cpp

View Options

src/test/script_tests.cpp

View Options

src/test/txvalidationcache_tests.cpp

API rename: Sign->SignECDSA, Verify->VerifyECDSAClosedPublicActions

Details

Diff Detail

Event Timeline

Revision ContentsChangeset List

Diff 6757

src/key.h

src/key.cpp

src/pubkey.h

src/pubkey.cpp

src/script/interpreter.cpp

src/script/sign.cpp

src/test/checkdatasig_tests.cpp

src/test/key_tests.cpp

src/test/multisig_tests.cpp

src/test/script_tests.cpp

src/test/txvalidationcache_tests.cpp

API rename: Sign->SignECDSA, Verify->VerifyECDSA
ClosedPublic
Actions

Revision Contents
Changeset List