mikecrowe added a comment.

If this sort of check is deemed acceptable then I have plans for future 
improvements, including:

1. Automatically turning absl::StrFormat into fmt::format too (with an option 
to choose an alternative to `absl::StrFormat`.)

2. Detecting format strings that end in `"\n"` and turning the call into 
`std::println` rather than `std::print`.

3. It ought to be possible to do something with %m by using `std::error_code`.

4. Support for member functions and even `operator()` for help with converting 
logging class methods.

My attempts to do something sensible with wide characters failed miserably. 
I've never used wprintf etc. myself and it would probably require the skills of 
someone more familiar with them.

Since many other clang-tidy checks are for Abseil, support for `absl::PrintF` 
and `absl::FPrintF` is built in. They could be removed since it would always be 
possible to replace them using the provided options.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D149280

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to