diff options
| author | fanquake <fanquake@gmail.com> | 2021-12-02 09:21:56 +0800 |
|---|---|---|
| committer | fanquake <fanquake@gmail.com> | 2021-12-02 10:09:42 +0800 |
| commit | df562d698a386166ef93d03326c0480ea9bc11fe (patch) | |
| tree | 70f9f752fd7b167c130dd4a11f078791e0ba8a69 /src/net_permissions.cpp | |
| parent | 4633199cc8a466b8a2cfa14ba9d7793dd4c469f4 (diff) | |
| parent | fa46ac4d9d6dc99572c44c42adc9bc3f41d701d4 (diff) | |
Merge bitcoin/bitcoin#23637: miner: Remove uncompiled MTP code
fa46ac4d9d6dc99572c44c42adc9bc3f41d701d4 miner: Remove uncompiled MTP code (MarcoFalke)
fa6b7adf965e68ec6a46d867966b1d44af98e908 style: Add {} to if-bodies in node/miner (MarcoFalke)
Pull request description:
This removes uncompiled code.
Can be checked by inserting `static_assert(STANDARD_LOCKTIME_VERIFY_FLAGS & LOCKTIME_MEDIAN_TIME_PAST)` and compiling or by reading the source code.
Even if the code was compiled, it would be unsafe to execute, since it is not allowed to include transactions that are locked until some time after the current MTP.
Also, rename the member to cause explicit merge conflicts in case there is a patch out there referencing the variable.
ACKs for top commit:
shaavan:
ACK fa46ac4d9d6dc99572c44c42adc9bc3f41d701d4
theStack:
Code-review ACK fa46ac4d9d6dc99572c44c42adc9bc3f41d701d4
Tree-SHA512: 0288f45918996b58d0c0060773aa3cb15c828a649439f3d589c5d6b4854d6da1d8c2ea11d5ca06c654532453ab5ce1892de7ca820e284e96e78b959ef87cac5c
Diffstat (limited to 'src/net_permissions.cpp')
0 files changed, 0 insertions, 0 deletions