diff options
| author | W. J. van der Laan <laanwj@protonmail.com> | 2021-10-07 15:26:58 +0200 |
|---|---|---|
| committer | W. J. van der Laan <laanwj@protonmail.com> | 2021-10-07 15:41:57 +0200 |
| commit | 991753e4d50ea5c973f4d3330e5afba797b1b1e7 (patch) | |
| tree | 09fbf6bf32d81c32bbbd5c24b7710a4c8a6a6312 /src/support | |
| parent | 6334ff7364e23e883c40a7ed234a467fcd08fd3b (diff) | |
| parent | 429b49378ee3a3d73abe276cfd176c1ca08bf9b9 (diff) | |
| download | bitcoin-991753e4d50ea5c973f4d3330e5afba797b1b1e7.tar.xz | |
Merge bitcoin/bitcoin#23118: test: refactor: introduce `script_util` helper for creating P2PK scripts
429b49378ee3a3d73abe276cfd176c1ca08bf9b9 test: introduce script_util helper for creating P2PK scripts (Sebastian Falbesoner)
Pull request description:
This PR is a follow-up to #22363, which took use of already existing `script_util` helpers to get rid of manual CScript for the P2{PKH,SH,WPKH,WSH} output types, in order to increase readability and maintainability of the test code. Here the same is done for P2PK scripts by introducing a helper `key_to_p2pk_script` and using it. Note that the helper only accepts ECDSA pubkeys (i.e. ones with a size of 33 or 65 bytes), hence it can't be used for scripts in the form of [x-only-pubkey, OP_CHECKSIG].
ACKs for top commit:
brunoerg:
Code review ACK 429b49378ee3a3d73abe276cfd176c1ca08bf9b9
laanwj:
Code review ACK 429b49378ee3a3d73abe276cfd176c1ca08bf9b9
rajarshimaitra:
Concept + tACK https://github.com/bitcoin/bitcoin/pull/23118/commits/429b49378ee3a3d73abe276cfd176c1ca08bf9b9
Tree-SHA512: 984aea01eba5f38a328d69905d90a3a36f0a02419ca3e5baf3c8095895fb094e3780c7da16fad5851db3847bdb05ce8cda244ab09b79b8aa9602dfb3c5e0414c
Diffstat (limited to 'src/support')
0 files changed, 0 insertions, 0 deletions