diff options
| author | Wladimir J. van der Laan <laanwj@protonmail.com> | 2020-04-22 14:23:10 +0200 |
|---|---|---|
| committer | Wladimir J. van der Laan <laanwj@protonmail.com> | 2020-04-22 14:23:56 +0200 |
| commit | 9e8e813df5f19ff154d9314f2d039eb6153f06c4 (patch) | |
| tree | 57cd0d2f11f9ee65c9e741ac8bdd380fd9d2ddee /ci/test/00_setup_env.sh | |
| parent | acb4fa07416f37cfcf77dcafb26612845fd531a6 (diff) | |
| parent | 21fa0a44abe8c1b5c452e097eab20cf0ae988805 (diff) | |
| download | bitcoin-9e8e813df5f19ff154d9314f2d039eb6153f06c4.tar.xz | |
Merge #18410: Docs: Improve commenting for coins.cpp|h
21fa0a44abe8c1b5c452e097eab20cf0ae988805 [docs] use consistent naming for possible_overwrite (John Newbery)
2685c214cce4b07695273503e60350e3f05fe3e2 [tests] small whitespace fixup (John Newbery)
e9936966c08bd8a6ac02828131f619ddaa1ced13 scripted-diff: Rename PRUNED to SPENT in coins tests (John Newbery)
c205979031ff4e8e32a5f05bae813405f233fccd [docs] Improve commenting in coins.cpp|h (John Newbery)
Pull request description:
- Add full commenting for spentness / DIRTYness / FRESHness and which combinations are valid
- Remove the 'pruned' terminology, which doesn't make sense since per-txout chainstate db was merged (#10195).
- Rename `potential_overwrite` to `possible_overwrite` to standardize terminology (there were previously examples of both, which made searching the codebase difficult).
- Make other minor improvements to the comments
ACKs for top commit:
jonatack:
Re-ACK 21fa0a4 per `git diff 98bee55 21fa0a4` the only change since my previous review is the following code commenting diff in `src/coins.cpp::L177-179`; rebuilt/ran unit tests anyway as a sanity check on the unit test changes.
Tree-SHA512: 391e01588ef5edb417250080cec17361f982c4454bc5f8c6d78bbd528c68a2bb94373297760691295c24660ce1022ad3ef7599762f736c8eed772ce096d38c3d
Diffstat (limited to 'ci/test/00_setup_env.sh')
0 files changed, 0 insertions, 0 deletions