diff options
author | Samuel Dobson <dobsonsa68@gmail.com> | 2021-09-02 13:05:34 +1200 |
---|---|---|
committer | Samuel Dobson <dobsonsa68@gmail.com> | 2021-09-02 13:31:10 +1200 |
commit | 9a86327512b3f41d2523457293c95ae2a2448c35 (patch) | |
tree | 57af4127f3f4bcf136c74ac8d08e4e19048aed24 /contrib | |
parent | dc9ffb6bc4520b107653e6ff182a6f54de10073f (diff) | |
parent | d9d3ec07cfe45cfa55028cc879dc8a55aecb4d3c (diff) | |
download | bitcoin-9a86327512b3f41d2523457293c95ae2a2448c35.tar.xz |
Merge bitcoin/bitcoin#22512: Consolidate XOnlyPubKey lookup hack
d9d3ec07cfe45cfa55028cc879dc8a55aecb4d3c Consolidate XOnlyPubKey lookup hack (Andrew Chow)
Pull request description:
The places where we need to lookup information for a XOnlyPubKey
currently implement a hack which makes both serializations of the full
pubkey in order to try the CKeyIDs for the lookup functions. Instead of
duplicating this everywhere it is needed, we can consolidate the CKeyID
generation into a function, and then have wrappers around GetPubKey,
GetKey, and GetKeyOrigin which takes the XOnlyPubKey, retrieves all of
the CKeyIDs (using the new GetKeyIDs() function in XOnlyPubKey), and
tries their respective underlying lookup function.
Split from #22364
ACKs for top commit:
S3RK:
Code Review reACK d9d3ec0
Zero-1729:
re-crACK d9d3ec0
theStack:
re-ACK d9d3ec07cfe45cfa55028cc879dc8a55aecb4d3c
meshcollider:
Code review + functional test run ACK d9d3ec07cfe45cfa55028cc879dc8a55aecb4d3c
Tree-SHA512: 21a7f6d37fad74483a38006f82b3558337fe9ed30e0b4392e6fff82c22251a42ac996b43f06cdaa9289ee34a768e181d87aa4208b5538e36ae4977954e1fa6a0
Diffstat (limited to 'contrib')
0 files changed, 0 insertions, 0 deletions