cleanup db.cpp/.h after Ultraprune patch #2011

pull Diapolo wants to merge 1 commits into bitcoin:master from Diapolo:cleanup_db changing 2 files +8 −94
  1. Diapolo commented at 4:25 PM on November 14, 2012: none
    • remove obsolete functions, variables, definitions and such
    • cleanup layout (no need to keep that many ugly empty lines IMHO)

    I searched trough the source to see where functions are used and removed all that were not used anywhere anymore. Tried a compilation afterwards and the client still works fine :). Needs a careful review from a core dev.

  2. cleanup db.cpp/.h after Ultraprune patch
    - remove obsolete functions, variables, definitions and such
    - cleanup layout (no need to keep that many ugly empty lines IMHO)
    0c7d18cc0c
  3. sipa commented at 4:27 PM on November 14, 2012: member

    I think you're a bit over-eager, and missed the fact that mocking is only used in tests.

  4. Diapolo closed this on Nov 14, 2012

  5. Diapolo commented at 4:32 PM on November 14, 2012: none

    Indeed... searched through the client source but didn't search within tests, sorry.

  6. HashUnlimited referenced this in commit f7f188305e on Apr 6, 2018
  7. KolbyML referenced this in commit e79fbccc97 on Dec 5, 2020
  8. DrahtBot locked this on Sep 8, 2021
Contributors

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-04-21 18:16 UTC

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