`BTC(Input|Output)ScriptType.Bech32` is redundant and probably broken

Question

`BTC(Input|Output)ScriptType.Bech32` is redundant and probably broken

mrnerdhair opened this issue 2 years ago · comments

Bech32 is an address encoding scheme, not a script type, and Bitcoin now uses it for more than one type of script too. Everywhere BTC(Input|Output)ScriptType.Bech32 is used, it should be SpendWitness instead.

Remove BTC(Input|Output)ScriptType.Bech32.

Fixed by #427.