These files change infrequently, and not much header shuffling is required.
We don't add everything in src/util/ yet, because IWYU makes some dubious suggestions, which I'm going to follow up with upstream.
Soon we'll swap src/util/xyz.cpp for just src/util/.
Concept ACK.
<!--e57a25ab6845829454e8d69fc972939a-->
The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.
<!--174a7506f384e20aa4161008e828411d-->
Reviewers, this pull request conflicts with the following ones:
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.
55 | + " src/util/moneystr.cpp"\ 56 | + " src/util/serfloat.cpp"\ 57 | + " src/util/spanparsing.cpp"\ 58 | + " src/util/strencodings.cpp"\ 59 | + " src/util/syserror.cpp"\ 60 | + " src/util/url.cpp"\
Actually, this PR affects more files, e.g., asmap.cpp, readwritefile.cpp etc. Should they be listed here as well?
IWYU generates spurious output for some .cpp files, which would require adding headers that I'm not sure are correct, so I've excluded them for now.
These files change infrequently, and not much header shuffling is required.
We don't add everything in src/util/ yet, because IWYU makes some
dubious suggestions, which I'm going to follow up with upstream.
ACK 07f2c25d04c39a0074e1d9ee1b24b3e359c8153f, I have reviewed the code and it looks OK, I agree it can be merged.
Suggesting to apply clang-format-diff.py as well.