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.

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

BIP340 updates: even R, new tags, small fixups, clarifications #982

Merged
merged 15 commits into from
Aug 27, 2020

Conversation

sipa
Copy link
Member

@sipa sipa commented Aug 27, 2020

Two semantics changes:

There are also a number of changes to the document that do not change semantics, but just improve clarity, or fix typos.

jonasnick and others added 15 commits July 18, 2020 20:14
- key prefixing means prefixing the message
- array indexing starts with 0
- 'Gennaro' is spelled with two n's
- has_even_y definition takes P as argument

Thanks to Alan Szepieniec for pointing out these issues.
Jacobi symbol can be confusing because it may suggest that the modulus is
composite.

Thanks to Alan Szepieniec for pointing out this issue.
Clarify security argument of x-only pubkeys
BIP-0340: Miscellaneous fixups
Switch BIP340 to even R tiebreaker
Clarify that Jacobian coordinates are the optimization
@sipa
Copy link
Member Author

sipa commented Aug 27, 2020

@luke-jr I'm happy to rebase this to remove merge commits, but I believe you prefer it in this form?

@luke-jr luke-jr merged commit 336f948 into bitcoin:master Aug 27, 2020
@luke-jr
Copy link
Member

luke-jr commented Aug 27, 2020

Yes, thanks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
4 participants