[PATCH] D143342: [clang-tidy] Support std::format and std::print in readability-redundant-string-cstr
Mike Crowe via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Thu Feb 16 13:57:28 PST 2023
mikecrowe marked 2 inline comments as done.
mikecrowe added a comment.
I ended up splitting out the std::format and std::print tests to their own file which meant that I didn't need to modify the existing `redundant-string-cstr.cpp` file in this commit. (Though of course I had to extract the <string> header in D144216 <https://reviews.llvm.org/D144216> first.) Let me know if you don't like the split.
I also slightly re-ordered the tests and removed a few incorrect comments that incorrectly survived the switch to using `forEachArgumentWithParam`.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D143342/new/
https://reviews.llvm.org/D143342
More information about the cfe-commits
mailing list