Separate core memory usage computation in core_memusage.h #6453

pull sipa wants to merge 1 commits into bitcoin:master from sipa:coremem changing 11 files +76 −99
  1. sipa commented at 5:48 pm on July 17, 2015: member

    Remove support for computing recursive memory usage from memusage.h, making it more easy to plug in other functions.

    Furthermore, move the computation of memory usage of core data structures from primitives/*, to reduce its dependencies, specifically on boost.

    Should fix #6441.

  2. laanwj added the label Refactoring on Jul 17, 2015
  3. in src/core_memusage.h: in 1fc0741ddd outdated
    57+
    58+static inline size_t RecursiveDynamicUsage(const CBlockLocator& locator) {
    59+    return memusage::DynamicUsage(locator.vHave);
    60+}
    61+
    62+#endif // BITCOIN_PRIMITIVES_TRANSACTION_H
    


    Diapolo commented at 2:57 pm on July 18, 2015:
    Nit: Wrong end comment :).

    sipa commented at 3:18 pm on July 20, 2015:
    Thanks, fixed!
  4. laanwj commented at 10:28 am on July 20, 2015: member
    utACK, looks like the good solution to me
  5. Separate core memory usage computation in core_memusage.h 9e38d0f745
  6. sipa force-pushed on Jul 20, 2015
  7. laanwj merged this on Jul 20, 2015
  8. laanwj closed this on Jul 20, 2015

  9. laanwj referenced this in commit 8fe5cce3d5 on Jul 20, 2015
  10. zkbot referenced this in commit 00e59e5935 on Mar 23, 2017
  11. DrahtBot 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: 2024-10-04 19:12 UTC

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