aboutsummaryrefslogtreecommitdiff
path: root/CONTRIBUTING.md
diff options
context:
space:
mode:
authorAndrew Chow <github@achow101.com>2023-06-13 10:23:51 -0400
committerAndrew Chow <github@achow101.com>2023-06-13 10:37:25 -0400
commit58b36fc303df604bbfb9a752a58262b51c26bab9 (patch)
treeba573d05d609d0a8a557fa761927c1c15415ab46 /CONTRIBUTING.md
parent8de9bb7a5ab04669369e2bb59ea92a5c1a91a8d2 (diff)
parent3ef756a5b558a1dd2fcb93bc0d4237707aa04f3f (diff)
downloadbitcoin-58b36fc303df604bbfb9a752a58262b51c26bab9.tar.xz
Merge bitcoin/bitcoin#23962: Use `int32_t` type for most transaction size/weight values
3ef756a5b558a1dd2fcb93bc0d4237707aa04f3f Remove txmempool implicit-integer-sign-change sanitizer suppressions (Hennadii Stepanov) d2f6d2a95a9f6c1632c1ed3b5b5b67a49eb71d6b Use `int32_t` type for most transaction size/weight values (Hennadii Stepanov) Pull request description: From bitcoin/bitcoin#23957 which has been incorporated into this PR: > A file-wide suppression is problematic because it will wave through future violations, potentially bugs. > > Fix that by using per-statement casts. > > This refactor doesn't change behavior because the now explicit casts were previously done implicitly. > > Similar to commit 8b5a4de904b414fb3a818732cd0a2c90b91bc275 ACKs for top commit: achow101: ACK 3ef756a5b558a1dd2fcb93bc0d4237707aa04f3f 0xB10C: ACK 3ef756a5b558a1dd2fcb93bc0d4237707aa04f3f. I've focused my testing and code review on the tracepoint related changes. The docs, the test, and the mempool_monitor.py demo script are updated. I ran the `interface_usdt_mempool.py` test and the `mempool_monitor.py` script. The `mempool_monitor.py` output looks correct. Xekyo: codereview ACK 3ef756a5b558a1dd2fcb93bc0d4237707aa04f3f ryanofsky: Code review ACK 3ef756a5b558a1dd2fcb93bc0d4237707aa04f3f. Since last review, just rebased with more type changes in test and tracing code Tree-SHA512: 397407f72165b6fb85ff1794eb1447836c4f903efed1a05d7a9704c88aa9b86f330063964370bbd59f6b5e322e04e7ea8e467805d58dce381e68f7596433330f
Diffstat (limited to 'CONTRIBUTING.md')
0 files changed, 0 insertions, 0 deletions