Skip to content

BIP-174: Removing PSBT_OUT_TAP_LEAF_SCRIPT#1241

Merged
kallewoof merged 2 commits intobitcoin:masterfrom
dr-orlovsky:patch-11
Jan 7, 2022
Merged

BIP-174: Removing PSBT_OUT_TAP_LEAF_SCRIPT#1241
kallewoof merged 2 commits intobitcoin:masterfrom
dr-orlovsky:patch-11

Conversation

@dr-orlovsky
Copy link
Copy Markdown
Contributor

PSBT_OUT_TAP_LEAF_SCRIPT seemed to appear in output key sections by copy-paste from input section. First, it shares the same byte no as PSBT_OUT_TAP_TREE, second its description talks about "witness"

`PSBT_OUT_TAP_LEAF_SCRIPT` seemed to appear in output key sections by copy-paste from input section. First, it shares the same byte no as `PSBT_OUT_TAP_TREE`, second its description talks about "witness"
@achow101
Copy link
Copy Markdown
Member

ACK 457e354

@achow101
Copy link
Copy Markdown
Member

achow101 commented Dec 9, 2021

While we're making fixups like this, can you also remove PSBT_GLOBAL_SIGHASH_SINGLE_INPUT? That's not part of BIP 370 anymore, and so shouldn't be listed in 174.

@kallewoof
Copy link
Copy Markdown
Contributor

Didn't realize this was RFM. Will wait for requested change.

@dr-orlovsky
Copy link
Copy Markdown
Contributor Author

@achow101 sorry for taking that so long. Removed according to your suggestion

@achow101
Copy link
Copy Markdown
Member

achow101 commented Jan 6, 2022

ACK d165229

@kallewoof kallewoof merged commit 3693cdf into bitcoin:master Jan 7, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants