[PATCH] D140968: [clang-tidy] Add check for passing the result of `std::string::c_str` to `strlen`

Alex Coster via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Jan 5 02:33:38 PST 2023


acoster marked 12 inline comments as done.
acoster added inline comments.


================
Comment at: clang-tools-extra/docs/clang-tidy/checks/readability/strlen-string-cstr.rst:24
+
+.. option:: EnableForDataMethod
+
----------------
carlosgalvezp wrote:
> Is there a use case for wanting this option? (As opposed to unconditionally warning about data()) The problem is the same and the same fix applies?
@carlosgalvezp Removed the option.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D140968



More information about the cfe-commits mailing list