Eugene.Zelenko added inline comments.
================ Comment at: clang-tools-extra/docs/clang-tidy/checks/modernize/use-std-format.rst:6 + +Converts calls to ``absl::StrFormat`` to equivalent calls to C++20's +``std::format`` function, modifying the format string appropriately. The ---------------- Please synchronize first statement with Release Notes. ================ Comment at: clang-tools-extra/docs/clang-tidy/checks/modernize/use-std-format.rst:58 + of -42 and the signed representation of 0xffffffff (often 4294967254 + and -1 respectively.) When false (which is the default), these casts + will not be added which may cause a change in the output. Note that this ---------------- Please highlight `false` with single back-ticks. ================ Comment at: clang-tools-extra/docs/clang-tidy/checks/modernize/use-std-format.rst:70 + immediately afterwards. The defaualt value for this option is + ``absl::StrFormat``. + ---------------- Single back-ticks for option value. ================ Comment at: clang-tools-extra/docs/clang-tidy/checks/modernize/use-std-format.rst:84 + `ReplacementFormatFunction` so that a ``#include`` directive can be added if + required. If `ReplacementFormatFunction` is ``std::format`` then this option will + default to ``<format>``, otherwise this option will default to nothing ---------------- Ditto. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D154287/new/ https://reviews.llvm.org/D154287 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
