diff options
| author | MarcoFalke <falke.marco@gmail.com> | 2020-07-09 12:54:03 +0200 |
|---|---|---|
| committer | MarcoFalke <falke.marco@gmail.com> | 2020-07-09 12:54:17 +0200 |
| commit | c4a44186d816716f5918fd23464a51033f208b71 (patch) | |
| tree | f22758bce27765ee4e8daf0a45711de6d9369d76 /test/functional/wallet_dump.py | |
| parent | f7c19e829eca10ce8b4acafc61264f8bb9b922f3 (diff) | |
| parent | e846a2a1d9e8aa37abfa55bd4d2a0a9f55ba6c8e (diff) | |
| download | bitcoin-c4a44186d816716f5918fd23464a51033f208b71.tar.xz | |
Merge #19085: Refactor: clean up PeriodicFlush()
e846a2a1d9e8aa37abfa55bd4d2a0a9f55ba6c8e refactor: clean up PeriodicFlush() (John Newbery)
Pull request description:
`PeriodicFlush()` is much more convoluted than it needs to be: it has triple nesting, local variables counting refs and return values, and increments the `mapFileUseCount` iterator unnecessarily. Removing all of that makes the function much easier to understand.
ACKs for top commit:
MarcoFalke:
ACK e846a2a1d9e8aa37abfa55bd4d2a0a9f55ba6c8e 🎁
jonatack:
re-ACK e846a2a per `git range-diff f7c19e8 7c10020 e846a2a`
promag:
ACK e846a2a1d9e8aa37abfa55bd4d2a0a9f55ba6c8e.
Tree-SHA512: 22bc600a5268b139c0a2c16b5a9f14837b262670ec24aef00643fcedd1c3ebcbf46dea1633e76adc8acf78e8840b776e17127307c5ee95308caa94239dad5b88
Diffstat (limited to 'test/functional/wallet_dump.py')
0 files changed, 0 insertions, 0 deletions