kernel: remove util/bytevectorhash.cpp #26237

pull fanquake wants to merge 1 commits into bitcoin:master from fanquake:kernel_drop_byte_vector_hash changing 1 files +0 −1
  1. fanquake commented at 7:27 pm on October 3, 2022: member
    This is no-longer used.
  2. kernel: remove util/bytevectorhash.cpp 4bee62e9b8
  3. fanquake requested review from theuni on Oct 3, 2022
  4. theuni commented at 8:17 pm on October 3, 2022: member

    Concept ACK. What’s the history here though, what freed this up?

    Edit: looks like it was never actually needed in the kernel?

  5. DrahtBot added the label Validation on Oct 3, 2022
  6. fanquake commented at 8:22 am on October 4, 2022: member

    Concept ACK. What’s the history here though, what freed this up? Edit: looks like it was never actually needed in the kernel?

    Probably forgotten about during #24410, which extracted the indexes.

  7. hebasto approved
  8. hebasto commented at 8:35 am on October 4, 2022: member
    ACK 4bee62e9b8fb06749b5521d717a475192886a45d, I have reviewed the code and it looks OK, I agree it can be merged.
  9. MarcoFalke merged this on Oct 4, 2022
  10. MarcoFalke closed this on Oct 4, 2022

  11. fanquake deleted the branch on Oct 4, 2022
  12. sidhujag referenced this in commit ae56cdd9dd on Oct 4, 2022
  13. bitcoin locked this on Oct 4, 2023


fanquake theuni hebasto


theuni

Labels
Validation


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-12-21 15:12 UTC

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