[PATCH] D143342: [clang-tidy] Support std::format and std::print in readability-redundant-string-cstr
Piotr Zegar via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Sat Mar 11 10:22:38 PST 2023
PiotrZSL added a comment.
"Do you think that the change can land like in its current state first? Or would you prefer that the configuration option is added at the same time?"
Code is fine, probably If would would write this, then I would bother to split into C++20 and C++2B simply because `std::print` wouldn't compile if it wouldn't be available. So I would just use hasAnyName.
But code looks fine, configuration could be added later...
CHANGES SINCE LAST ACTION
More information about the cfe-commits