[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
Sat Mar 11 10:28:18 PST 2023


mikecrowe added a comment.

In D143342#4186809 <https://reviews.llvm.org/D143342#4186809>, @PiotrZSL wrote:

> 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...

Thanks. I originally wrote it like that, but @njames93 requested that I checked the version, so I did. :) It doesn't do any harm and might avoid confusion.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D143342/new/

https://reviews.llvm.org/D143342



More information about the cfe-commits mailing list