utils refactor: strencode changes #17816

pull sanjaykdragon wants to merge 5 commits into bitcoin:master from sanjaykdragon:strencode_changes changing 2 files +133 −130
  1. sanjaykdragon commented at 6:14 PM on December 28, 2019: contributor

    Changed a few things in strencodings.cpp: caf91fa and 99d84fc shouldn't be part of this branch, forgot to separate. the real changes are in: 851ceb0 - main refactoring 752cc25 - reverted a bit of ^ (because we are not including boost libs in this area) 9ba9a14 - nothing major, I just messed up spacing on something

  2. consensus refactor: changed for loops to range based for loops (modern cpp).
    Should not change behavior of consensus itself.
    caf91fac90
  3. fixed some stuff, im an idiot 99d84fcd6a
  4. utils refactor: updated atoi, i64tostr, itostr 851ceb0e9b
  5. utils refactor: updated atoi, i64tostr, itostr to more modern cpp. More range based for loop, also started using str::empty(). Removed <algorithm> header, seemingly not needed. 752cc25275
  6. keep p_util_hexdigit compacted 9ba9a14cab
  7. sanjaykdragon closed this on Dec 28, 2019

  8. DrahtBot locked this on Dec 16, 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-05-02 21:14 UTC

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