doc: follow-ups from #23365 (index backwards search) #23777

pull mzumsande wants to merge 1 commits into bitcoin:master from mzumsande:202112_index_followups changing 2 files +4 −0
  1. mzumsande commented at 9:50 pm on December 14, 2021: member

    This adds two explanations suggested in the review of #23365, that I didn’t manage to address before that PR was merged:

    #23365 (review) #23365 (review)

  2. DrahtBot added the label UTXO Db and Indexes on Dec 14, 2021
  3. fanquake requested review from jamesob on Dec 14, 2021
  4. fanquake requested review from jnewbery on Dec 14, 2021
  5. fanquake added the label Docs on Dec 14, 2021
  6. in src/index/base.cpp:100 in 43ccaf6fdf outdated
     95@@ -96,6 +96,8 @@ bool BaseIndex::Init()
     96                     prune_violation = false;
     97                     break;
     98                 }
     99+                // block->pprev must exist at this point, since block_to_test is chosen such that it
    100+                // must be encountered when going backwards from the tip
    


    MarcoFalke commented at 8:40 am on December 15, 2021:
    0                // block->pprev must exist at this point, since block_to_test is part of the chain and thus
    1                // must be encountered when going backwards from the tip
    

    mzumsande commented at 10:06 am on December 15, 2021:
    done, thanks
  7. MarcoFalke approved
  8. MarcoFalke commented at 8:40 am on December 15, 2021: member
    ACK, left a nit (can be ignored)
  9. doc: add explanations for assert in index and magic numbers in test
    These were suggested in review of #23365
    
    Co-authored-by: John Newbery <john@johnnewbery.com>
    e4a8d561ed
  10. mzumsande force-pushed on Dec 15, 2021
  11. jnewbery commented at 11:18 am on December 15, 2021: member
    ACK e4a8d561edf3cfb326e86c87155fed41a61e7333
  12. MarcoFalke merged this on Dec 16, 2021
  13. MarcoFalke closed this on Dec 16, 2021

  14. sidhujag referenced this in commit 8eeb7d1aaf on Dec 16, 2021
  15. mzumsande deleted the branch on Dec 16, 2021
  16. Fabcien referenced this in commit f929502483 on Apr 5, 2022
  17. DrahtBot locked this on Dec 16, 2022

github-metadata-mirror

This is a metadata mirror of the GitHub repository bitcoin/bitcoin. This site is not affiliated with GitHub. Content is generated from a GitHub metadata backup.
generated: 2024-11-21 09:12 UTC

This site is hosted by @0xB10C
More mirrored repositories can be found on mirror.b10c.me