refactor: Move txmempool_entry.h –> kernel/mempool_entry.h #26609

pull hebasto wants to merge 1 commits into bitcoin:master from hebasto:221130-move changing 16 files +20 −20
  1. hebasto commented at 10:41 am on November 30, 2022: member

    This PR addresses the #17786 (review):

    why not move it to the right place, that is to kernel/txmempool_entry.h?

  2. refactor: Move `txmempool_entry.h` --> `kernel/mempool_entry.h` 38941a703e
  3. DrahtBot commented at 10:41 am on November 30, 2022: contributor

    The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.

    Reviews

    See the guideline for information on the review process.

    Type Reviewers
    ACK MarcoFalke

    If your review is incorrectly listed, please react with 👎 to this comment and the bot will ignore it on the next update.

    Conflicts

    Reviewers, this pull request conflicts with the following ones:

    • #26590 (net, refactor: Kill proxy globals by dergoegge)
    • #24539 (Add a “tx output spender” index by sstone)

    If you consider this pull request important, please also help to review the conflicting pull requests. Ideally, start with the one that should be merged first.

  4. DrahtBot added the label Refactoring on Nov 30, 2022
  5. hebasto added the label Mempool on Dec 2, 2022
  6. glozow requested review from maflcko on Dec 6, 2022
  7. glozow commented at 5:21 pm on December 6, 2022: member
    Makes sense to me, given src/kernel/mempool_limits.h is there
  8. maflcko commented at 6:04 pm on December 6, 2022: member

    review ACK 38941a703e079709bb465ad1fcde50e11350f8f1 📊

    Signature:

     0-----BEGIN PGP SIGNED MESSAGE-----
     1Hash: SHA512
     2
     3review ACK 38941a703e079709bb465ad1fcde50e11350f8f1 📊
     4-----BEGIN PGP SIGNATURE-----
     5
     6iQGzBAEBCgAdFiEE+rVPoUahrI9sLGYTzit1aX5ppUgFAlwqrYAACgkQzit1aX5p
     7pUhB+Av+Mjyg97YXKq3Wn9HWq8YhVBCwYg2ryPwjthV16MJpoYA0pWdxqPPPcZK2
     8z6kSh2WRAVjyqegez70y0sBo1WmSeR9BABOTP0/OHe81XK2ku7VIXt6TIDIqbu3B
     9/J3TVJ+sEkR5RKyu8ydi0PlmoS0GNrc/a7osieRbpySQz1jaxus1x680qa5Zulz2
    103ASaDcUEzDa+6SZ9zb38v21z3FS8IEdTb3MFiLv3eoPtSxeRT0PaLUQgqB24bk3w
    116TKOnWWpjPMCc1CGrNI22/1tGIj9Jduy36DZYuAsgKzhqOfpPuaJ2YqBgBreIpTG
    126VmkwuIguV5MPQ0GFJkPVlhLc9wUZX0sMeJW3OAktQxj8LzCWP8tlkMnvCJkDEkd
    13FHFmuUENuxO5tPt8vO9Ru4DCp+7p36jFhwbO3FIPgX35B3mMDOExARAgSx42Ut0B
    14qRLsvBWZY09H0crkidL94M6OitoKbzR4HUtZSSeSp14ikOnnOFAJhBTxVWkOWGCA
    15STW1iLGl
    16=4JVU
    17-----END PGP SIGNATURE-----
    
  9. maflcko merged this on Dec 6, 2022
  10. maflcko closed this on Dec 6, 2022

  11. hebasto deleted the branch on Dec 6, 2022
  12. sidhujag referenced this in commit 98492d5cff on Dec 6, 2022
  13. bitcoin locked this on Dec 6, 2023

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-24 09:12 UTC

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