util: Move ResolveErrMsg to util/error #16620

pull MarcoFalke wants to merge 1 commits into bitcoin:master from MarcoFalke:1908-utilErrorResolveErrMsg changing 4 files +13 −10
  1. MarcoFalke commented at 2:08 pm on August 15, 2019: member
    Pull request #16248 (review) duplicated the body of this util function. The whole point of the util function is to be shared, so do that here as a fixup to #16248
  2. util: Move ResolveErrMsg to util/error fa27c55b05
  3. MarcoFalke added the label Refactoring on Aug 15, 2019
  4. Sjors commented at 2:13 pm on August 15, 2019: member
    utACK fa27c55
  5. DrahtBot commented at 8:03 pm on August 15, 2019: member

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

    Conflicts

    Reviewers, this pull request conflicts with the following ones:

    • #16224 (gui: Bilingual GUI error messages 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. in src/init.cpp:825 in fa27c55b05
    821@@ -822,11 +822,6 @@ void InitParameterInteraction()
    822     }
    823 }
    824 
    825-static std::string ResolveErrMsg(const char * const optname, const std::string& strBind)
    


    promag commented at 11:36 pm on August 15, 2019:
    Shoud #include <util/error.h> then?

    MarcoFalke commented at 11:52 am on August 16, 2019:
    Will leave as is for now to not invalidate review
  7. NicolasDorier commented at 4:27 am on August 16, 2019: contributor
    utACK seems like travis need a kick-in-the-box (and maybe need #16620 (review))
  8. ryanofsky approved
  9. ryanofsky commented at 10:54 am on August 16, 2019: member
    utACK fa27c55b0593c769b6ad87de0b59df3816d73548
  10. MarcoFalke merged this on Aug 16, 2019
  11. MarcoFalke closed this on Aug 16, 2019

  12. MarcoFalke referenced this in commit 21a165325e on Aug 16, 2019
  13. deadalnix referenced this in commit 35f36b8ef1 on Jul 28, 2020
  14. DrahtBot locked this on Dec 16, 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: 2024-11-17 09:12 UTC

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