[PATCH] D51061: [clang-tidy] abseil-str-cat-append

Eugene Zelenko via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Aug 22 10:15:43 PDT 2018


Eugene.Zelenko added inline comments.


================
Comment at: docs/ReleaseNotes.rst:70
+
+  Flags uses of ``absl::StrCat()`` to append to a std::string. Suggests 
+  ``absl::StrAppend()`` should be used instead.
----------------
Please enclose std::string into ``


================
Comment at: docs/ReleaseNotes.rst:72
+  ``absl::StrAppend()`` should be used instead.
+  Does not diagnose cases where ``abls::StrCat()`` is used as a template 
+  argument for a functor.
----------------
Details belongs to documentation.


https://reviews.llvm.org/D51061





More information about the cfe-commits mailing list