Use stricter ToIntegral over LocaleIndependentAtoi where possible #23184

pull MarcoFalke wants to merge 3 commits into bitcoin:master from MarcoFalke:2110-ToIntegral changing 7 files +22 −9
  1. MarcoFalke commented at 11:10 AM on October 5, 2021: member

    No need to use LocaleIndependentAtoi when a string is entirely composed of IsDigits and ToIntegral is sufficient.

  2. test: Add ParseMoney embedded whitespace tests fa2ad19a5e
  3. doc: Fixup ToIntegral docs faae83c8f8
  4. Use stricter ToIntegral over LocaleIndependentAtoi where possible fa6a25e92d
  5. MarcoFalke cross-referenced this on Oct 5, 2021 from issue util: Replace use of locale dependent atoi(…) with locale-independent std::from_chars(…) (C++17) by practicalswift
  6. practicalswift commented at 11:14 AM on October 5, 2021: contributor

    Concept ACK

  7. MarcoFalke commented at 11:21 AM on October 5, 2021: member

    nvm.

    This is not a refactor.

  8. MarcoFalke closed this on Oct 5, 2021
  9. MarcoFalke deleted the branch on Oct 5, 2021
  10. bitcoin locked this on Oct 30, 2022
Contributors
MarcoFalkepracticalswift
Linked (view graph)
#20452 util: Replace use of locale dependent atoi(…) with locale-independent std::from_chars(…) (C++17)
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-05-19 06:53 UTC