Skip to content

BIP174: Fix wrong description about Proprietary Use Type#860

Merged
luke-jr merged 1 commit intobitcoin:masterfrom
azuchi:fix-wrong-description-bip174
Dec 13, 2019
Merged

BIP174: Fix wrong description about Proprietary Use Type#860
luke-jr merged 1 commit intobitcoin:masterfrom
azuchi:fix-wrong-description-bip174

Conversation

@azuchi
Copy link
Copy Markdown
Contributor

@azuchi azuchi commented Nov 14, 2019

About BIP-174, PSBT_GLOBAL_PROPRIETARY, PSBT_INPUT_PROPRIETARY, PSBT_OUTPUT_PROPRIETARY key description is incorrectly described as Version Number, so it fix to Proprietary Use Type.

@junderw
Copy link
Copy Markdown
Contributor

junderw commented Nov 14, 2019

ACK

@achow101
Copy link
Copy Markdown
Member

ACK 87a8540

That's what I get for copy-pasting.

@maflcko
Copy link
Copy Markdown
Member

maflcko commented Nov 19, 2019

@luke-jr

@dgpv
Copy link
Copy Markdown
Contributor

dgpv commented Dec 2, 2019

Shouldn't the names for proprietary input and output types be PSBT_IN_PROPRIETARY, PSBT_OUT_PROPRIETARY, to be consistent with other input/output type names ?

@dgpv
Copy link
Copy Markdown
Contributor

dgpv commented Dec 2, 2019

(Also it seems that proprietary types are missing from Appendix A)

@luke-jr luke-jr merged commit 0a388fa into bitcoin:master Dec 13, 2019
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.

6 participants