HomePhabricator

Remove the boost/algorithm/string/case_conv.hpp dependency

Description

Remove the boost/algorithm/string/case_conv.hpp dependency

Summary:

  • Implements ParseNetwork unit test.

This commit implements a unit test that validates the ParseNetwork(std::string) implementation in netbase.cpp.

  • Implements custom tolower and toupper functions.

This commit implements custom equivalents for the C and C++ tolower and toupper Standard Library functions.
In addition it implements a utility function to capitalize the first letter of a string.

  • Removes the Boost case_conv.hpp dependency.

This commit removes the boost/algorithm/string/case_conv.hpp dependency from the project. It replaces the boost::to_lower and boost::to_upper functions with custom functions that are locale independent and ASCII deterministic.

This is a backport of Core PR13671

Depends on D2824

Test Plan:

make check

Reviewers: #bitcoin_abc, Fabien

Reviewed By: #bitcoin_abc, Fabien

Subscribers: Fabien

Differential Revision: https://reviews.bitcoinabc.org/D2825

Details

Provenance
251 <13120787+251Labs@users.noreply.github.com>Authored on Jul 15 2018, 19:45
deadalnixCommitted on Apr 19 2019, 14:26
deadalnixPushed on Apr 19 2019, 14:26
Reviewer
Restricted Project
Differential Revision
D2825: Remove the boost/algorithm/string/case_conv.hpp dependency
Parents
rABC8080bab0d20c: scripted-diff: Replace NET_TOR with NET_ONION
Branches
Unknown
Tags
Unknown

Event Timeline