diff options
| author | Andrew Chow <achow101-github@achow101.com> | 2022-03-14 10:27:08 +0000 |
|---|---|---|
| committer | Andrew Chow <achow101-github@achow101.com> | 2022-03-14 10:31:25 +0000 |
| commit | 25d045a9ecc24a2752685d176c28f8f75bb100f6 (patch) | |
| tree | 6e53052063c924f6b95388a26454eca845f619bc /src/test/miner_tests.cpp | |
| parent | e0881aa5f0cde01b0590770f3202dfaf0919e2d4 (diff) | |
| parent | fa8e76bb9002a126b3645bd9533c282f5dfff111 (diff) | |
Merge bitcoin/bitcoin#24225: wallet: Add sanity checks to DiscourageFeeSniping
fa8e76bb9002a126b3645bd9533c282f5dfff111 wallet: Add sanity checks to AntiFeeSnipe (MarcoFalke)
Pull request description:
I added those sanity checks as part of implementing BIP 326, but I think they make sense on their own. The checks require the transaction to be passed in to `DiscourageFeeSniping`. Also, replace `(int)locktime` cast with the equivalent `int(locktime)` cast.
ACKs for top commit:
chris-belcher:
ACK https://github.com/bitcoin/bitcoin/pull/24225/commits/fa8e76bb9002a126b3645bd9533c282f5dfff111
S3RK:
Code review ACK fa8e76bb9002a126b3645bd9533c282f5dfff111
achow101:
ACK fa8e76bb9002a126b3645bd9533c282f5dfff111
w0xlt:
Code Review ACK https://github.com/bitcoin/bitcoin/pull/24225/commits/fa8e76bb9002a126b3645bd9533c282f5dfff111
Tree-SHA512: 6fe37c85f074851ef09cae8addcb836257089290fecec515c129c3180e9ccb64740aaac76043af10ce7c213e5001568ca6b4b62ae9631f0994ed676b07118074
Diffstat (limited to 'src/test/miner_tests.cpp')
0 files changed, 0 insertions, 0 deletions