CCoinsViewErrorCatcher raison-d-etre #8015

pull ghost wants to merge 1 commits into bitcoin:master from changing 1 files +5 −0
  1. ghost commented at 4:16 AM on May 6, 2016: none

    Still not thrilled with the class ( i.e. init.cpp 1232-1234 ), but, hey, it may indeed be minimally invasive. Only @sipa would know whether "common" was the reason for preferring a wrapper class to a new function ( #5619 ). 'Tis my best guess.

  2. CCoinsViewErrorCatcher raison-d-etre a4d5855a25
  3. MarcoFalke added the label Docs and Output on May 6, 2016
  4. fanquake commented at 11:32 AM on May 12, 2016: member

    @sipa would you like to comment here?

  5. sipa commented at 10:49 PM on May 12, 2016: member

    Right, the reason is to avoid a dependency from the leveldb/database code on main.

  6. arowser commented at 8:43 AM on May 25, 2016: contributor

    Can one of the admins verify this patch?

  7. laanwj merged this on May 26, 2016
  8. laanwj closed this on May 26, 2016

  9. laanwj referenced this in commit 6fc6325f77 on May 26, 2016
  10. codablock referenced this in commit 70724c3440 on Sep 16, 2017
  11. codablock referenced this in commit 23ea4a998e on Sep 19, 2017
  12. codablock referenced this in commit b71ee087ad on Dec 21, 2017
  13. andvgal referenced this in commit b0b43e8e28 on Jan 6, 2019
  14. 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: 2026-04-17 15:15 UTC

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