refactor: Extract util/exception from util/system #26301

pull Empact wants to merge 1 commits into bitcoin:master from Empact:2022-10-util-exception changing 15 files +67 −27
  1. Empact commented at 7:19 PM on October 12, 2022: member

    This is the only use of std::exception in util/system, and paves the ways of further splitting that up to remove circular includes and in support of a minimal kernel.

    See overall effort, and prior conversation re motivation.

    Include libloaderapi.h for GetModuleFileNameA as that's the source noted here: https://docs.microsoft.com/en-us/windows/win32/api/libloaderapi/nf-libloaderapi-getmodulefilenamea

    Recommend using git diff master --color-moved=dimmed-zebra

  2. Empact commented at 7:20 PM on October 12, 2022: member
  3. Empact force-pushed on Oct 12, 2022
  4. DrahtBot added the label Refactoring on Oct 12, 2022
  5. DrahtBot commented at 10:35 PM on October 12, 2022: contributor

    <!--e57a25ab6845829454e8d69fc972939a-->

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

    <!--021abf342d371248e50ceaed478a90ca-->

    Reviews

    See the guideline for information on the review process. A summary of reviews will appear here.

    <!--174a7506f384e20aa4161008e828411d-->

    Conflicts

    Reviewers, this pull request conflicts with the following ones:

    • #26504 (Add UNREACHABLE macro and drop -Wreturn-type/C4715 warnings suppressions by hebasto)

    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.

  6. refactor: Extract util/exception from util/system
    This is a minimal extraction of a single public method, but also
    the only use of std::exception in util/system.
    
    Include libloaderapi.h for GetModuleFileNameA as that's the
    source noted here: https://docs.microsoft.com/en-us/windows/win32/api/libloaderapi/nf-libloaderapi-getmodulefilenamea
    f12cd1d77b
  7. Empact force-pushed on Jan 10, 2023
  8. Empact commented at 7:50 PM on January 10, 2023: member

    Closing in favor of one-shot #25152, as the incremental approach doesn't seem to be compelling attention.

  9. Empact closed this on Jan 10, 2023

  10. Empact deleted the branch on Mar 26, 2023
  11. bitcoin locked this on Mar 25, 2024

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-13 21:13 UTC

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