Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Move ECDSA pubkey recovery key to a module #291

Merged
merged 3 commits into from
Aug 28, 2015
Merged

Move ECDSA pubkey recovery key to a module #291

merged 3 commits into from
Aug 28, 2015

Conversation

sipa
Copy link
Contributor

@sipa sipa commented Aug 27, 2015

Builds on top of #289.

@sipa sipa changed the title Move ECDH pubkey recovery key to a module Move ECDSA pubkey recovery key to a module Aug 27, 2015
@apoelstra
Copy link
Contributor

ACK

@sipa
Copy link
Contributor Author

sipa commented Aug 27, 2015

Rebased.

@dcousens
Copy link
Contributor

concept ACK

@sipa
Copy link
Contributor Author

sipa commented Aug 27, 2015

Changed .travis.yml to actually try compiling/testing recovery in some cases.

@sipa sipa mentioned this pull request Aug 28, 2015
Closed
@gmaxwell
Copy link
Contributor

ACK

@sipa sipa merged commit 9f443be into bitcoin-core:master Aug 28, 2015
sipa added a commit that referenced this pull request Aug 28, 2015
@sipa
Merge pull request #291
5eb4356 
9f443be Move pubkey recovery code to separate module (Pieter Wuille)
d49abbd Separate ECDSA recovery tests (Pieter Wuille)
439d34a Separate recoverable and normal signatures (Pieter Wuille)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@sipa @apoelstra @dcousens @gmaxwell