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 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
