[PATCH] D149280: [clang-tidy] Add modernize-printf-to-std-print check

[Piotr Zegar via Phabricator via cfe-commits]

PiotrZSL added inline comments.


================
Comment at: clang-tools-extra/docs/clang-tidy/checks/modernize/use-std-print.rst:117
+
+.. option:: PrintFunction
+
----------------
mikecrowe wrote:
> PiotrZSL wrote:
> > mikecrowe wrote:
> > > Is `PrintFunction` (and the soon-to-arrive `PrintlnFunction`) distinct enough from `PrintfLikeFunctions` and `FprintfLikeFunctions`? Should I use `ReplacementPrintFunction` instead?
> > Use `Replacement`. It will be way better.
> I assume that you mean `ReplacementPrintFunction`. (I see `ReplacementString` used in other checks, so I think that just `Replacement` might be a bit vague.)
Yes, ... `ReplacementPrintFunction`


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D149280