Add tags to mempool's mapTx indices #7539

pull sdaftuar wants to merge 1 commits into bitcoin:master from sdaftuar:use-tags changing 4 files +24 −16
  1. sdaftuar commented at 5:39 PM on February 16, 2016: member

    This is a small refactor to how we use the boost multi_index in CTxMemPool -- rather than have to use fixed index numbers in our code, I added some tags so that we can use logical names instead, so that e.g. nth_index<2> becomes index<entry_time>.

    This was motivated by me rebasing some old work on to master, and I realized that the usage of fixed indices was error prone.

    I didn't bother with a tag name for the zero-th index (txid), since nothing seems to explicitly use it, but I can add one if people think that'd be better.

  2. Add tags to mempool's mapTx indices 086da92ea7
  3. laanwj added the label UTXO Db and Indexes on Feb 16, 2016
  4. laanwj added the label Mempool on Feb 16, 2016
  5. laanwj removed the label UTXO Db and Indexes on Feb 16, 2016
  6. sipa commented at 7:51 PM on February 16, 2016: member

    utACK

  7. dcousens commented at 10:23 PM on February 16, 2016: contributor

    utACK 086da92

  8. sdaftuar commented at 9:24 PM on February 23, 2016: member

    ping. If this looks fine to everyone it would be nice to have this merged, as it'll make some work I hope to PR soon on top of this somewhat cleaner.

  9. sdaftuar cross-referenced this on Feb 24, 2016 from issue Mempool: Add tracking of ancestor packages by sdaftuar
  10. luke-jr commented at 11:38 AM on February 25, 2016: member

    Concept ACK

  11. sipa merged this on Mar 5, 2016
  12. sipa closed this on Mar 5, 2016

  13. sipa referenced this in commit 9f33dba05c on Mar 5, 2016
  14. sickpig cross-referenced this on Mar 1, 2017 from issue [backport] Add tags to mempool's mapTx indices by kyuupichan
  15. kyuupichan cross-referenced this on Mar 4, 2017 from issue Backports to miner.cpp by kyuupichan
  16. kyuupichan cross-referenced this on Mar 19, 2017 from issue Miner backport by kyuupichan
  17. codablock referenced this in commit 24fea8c0f3 on Sep 16, 2017
  18. codablock referenced this in commit fc725874ae on Sep 19, 2017
  19. codablock referenced this in commit 19976f33f4 on Dec 9, 2017
  20. codablock referenced this in commit 00d42b7c90 on Dec 19, 2017
  21. MarkLTZ referenced this in commit c2255fd83d on Apr 28, 2019
  22. furszy cross-referenced this on Jul 4, 2020 from issue Up to CTransactionRef point back ports to do list. by furszy
  23. furszy cross-referenced this on Jul 4, 2020 from issue [WIP] Miner code reworked from upstream 6898 by furszy
  24. furszy cross-referenced this on Oct 12, 2020 from issue [Mempool] Add tags to mempool's mapTx indices by furszy
  25. furszy referenced this in commit 0a820797fb on Oct 22, 2020
  26. bitcoin locked this on Sep 8, 2021

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: 2026-05-19 06:55 UTC