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 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
