diff options
author | MarcoFalke <falke.marco@gmail.com> | 2022-04-06 11:12:05 +0200 |
---|---|---|
committer | MarcoFalke <falke.marco@gmail.com> | 2022-04-06 11:12:10 +0200 |
commit | 79bf1a0fa2c72911bb964f7303dd3acee3db584f (patch) | |
tree | 53e2666226671a59c455613310cd11aac77b39af /ci/test/00_setup_env_native_tidy.sh | |
parent | 372f1a3c25813cc79b5484599f0b26f857b99857 (diff) | |
parent | cccc4e879a8cb9d858a88ea46b28ea27ab79ca55 (diff) | |
download | bitcoin-79bf1a0fa2c72911bb964f7303dd3acee3db584f.tar.xz |
Merge bitcoin/bitcoin#24732: Remove buggy and confusing IncrementExtraNonce
cccc4e879a8cb9d858a88ea46b28ea27ab79ca55 Remove nHeightEnd and nHeight in generateBlocks helper (MarcoFalke)
fa38b1c8bd29e2c792737f6481ab928e46396b7e Remove buggy and confusing IncrementExtraNonce (MarcoFalke)
Pull request description:
IncrementExtraNonce has many issues:
* It is test-only code, but part of bitcoind
* It is using the block height of the tip, as opposed to the block's previous block as reference for the new height. See https://github.com/bitcoin/bitcoin/issues/24730#issuecomment-1085586193
* It has no use case in regtest testing. With a low difficulty the extra nonce won't be incremented. With a high difficulty the test-only functions are clumsy to handle anyway. For example, the generate* RPCs will return an empty array once they reached `maxtries`, as opposed to an error. Also the calls can't be aborted early unless the node shuts down completely. So I think it is fine to just remove the extra nonce functionality and leave it to the outside to implement, if needed. For example, a wrapper script can call the `generate*` RPCs once every second, to use the timestamp as extra nonce.
ACKs for top commit:
ajtowns:
ACK cccc4e879a8cb9d858a88ea46b28ea27ab79ca55
Tree-SHA512: d8a3989ad280ebd4b1b574159b3a396b8a42134347e6be3c88445162d86624d221c416456f45ae75aea62ed8c8a1a9bb3a2532924abca2ef7a879cb8e6b15654
Diffstat (limited to 'ci/test/00_setup_env_native_tidy.sh')
0 files changed, 0 insertions, 0 deletions