diff options
| author | MarcoFalke <[email protected]> | 2020-07-09 12:54:03 +0200 |
|---|---|---|
| committer | MarcoFalke <[email protected]> | 2020-07-09 12:54:17 +0200 |
| commit | c4a44186d816716f5918fd23464a51033f208b71 (patch) | |
| tree | f22758bce27765ee4e8daf0a45711de6d9369d76 /src/validationinterface.cpp | |
| parent | Merge #19320: wallet: Replace CDataStream& with CDataStream&& where appropriate (diff) | |
| parent | refactor: clean up PeriodicFlush() (diff) | |
| download | discoin-c4a44186d816716f5918fd23464a51033f208b71.tar.xz discoin-c4a44186d816716f5918fd23464a51033f208b71.zip | |
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 'src/validationinterface.cpp')
0 files changed, 0 insertions, 0 deletions