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

BIP 387: multi_a() descriptor #1567

Merged
merged 1 commit into from May 7, 2024
Merged

BIP 387: multi_a() descriptor #1567

merged 1 commit into from May 7, 2024

Conversation

achow101
Copy link
Member

No description provided.

bip-multi-a.mediawiki Outdated Show resolved Hide resolved
Copy link
Contributor

@jonatack jonatack left a 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.

bip-multi-a.mediawiki Outdated Show resolved Hide resolved
bip-multi-a.mediawiki Outdated Show resolved Hide resolved
bip-multi-a.mediawiki Outdated Show resolved Hide resolved
bip-multi-a.mediawiki Outdated Show resolved Hide resolved
bip-multi-a.mediawiki Outdated Show resolved Hide resolved
bip-multi-a.mediawiki Outdated Show resolved Hide resolved
bip-multi-a.mediawiki Outdated Show resolved Hide resolved
bip-multi-a.mediawiki Outdated Show resolved Hide resolved
@jonatack
Copy link
Contributor

Assigned BIP 387.

@achow101 achow101 changed the title BIP multi-a: multi_a() descriptor BIP 387: multi_a() descriptor Apr 22, 2024
README.mediawiki Outdated Show resolved Hide resolved
@achow101 achow101 force-pushed the multi_a branch 2 times, most recently from 1df09f4 to 4604171 Compare April 23, 2024 21:44
README.mediawiki Outdated Show resolved Hide resolved
bip-0387.mediawiki Outdated Show resolved Hide resolved
@achow101 achow101 force-pushed the multi_a branch 2 times, most recently from 01a467c to fe8b1cb Compare May 1, 2024 21:45
bip-0387.mediawiki Outdated Show resolved Hide resolved
bip-0387.mediawiki Outdated Show resolved Hide resolved
@jonatack
Copy link
Contributor

jonatack commented May 6, 2024

LGTM fe8b1cb modulo 2 nits above

Copy link
Contributor

@Christewart Christewart left a 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.
Copy link
Contributor

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.

Copy link
Member Author

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.)

@achow101
Copy link
Member Author

achow101 commented May 6, 2024

Are these implemented in the bitcoin core test suite?

Some are. I pulled a few from there, although I don't think there were any sortedmulti_a ones, so those are new in this BIP.

@jonatack
Copy link
Contributor

jonatack commented May 7, 2024

ACK 945b281

@jonatack jonatack merged commit c4c5c69 into bitcoin:master May 7, 2024
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
4 participants