-
Notifications
You must be signed in to change notification settings - Fork 5.5k
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
BIP 387: multi_a() descriptor #1567
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks like this needs a BIP number assignment? A few minor suggestions follow.
Assigned BIP 387. |
1df09f4
to
4604171
Compare
01a467c
to
fe8b1cb
Compare
LGTM fe8b1cb modulo 2 nits above |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I wont be able to implement the test for this anytime soon to verify they work. Are these implemented in the bitcoin core test suite?
|
||
==Abstract== | ||
|
||
This document specifies <tt>multi_a()</tt> and <tt>sortedmulti_a()</tt> output script descriptors. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
To ask the dumb question, what is a
? add
?
csa
would be clearer IMO.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The meaning of a
is left for the reader to decide :p
(I don't think it has any meaning, just something to distinguish from multi
. It's too late to change it since people are already using this.)
Some are. I pulled a few from there, although I don't think there were any |
ACK 945b281 |
No description provided.