[PATCH] D140307: [clang-tidy] Match derived types in in modernize-loop-convert

Carlos Galvez via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Jan 5 08:42:03 PST 2023


carlosgalvezp added inline comments.


================
Comment at: clang-tools-extra/clang-tidy/modernize/LoopConvertCheck.cpp:258
+                       hasMethod(cxxMethodDecl(hasName("begin"), isConst())),
+                       hasMethod(cxxMethodDecl(hasName("end"),
+                                               isConst())))))) // hasDeclaration
----------------
Replace tabs with spaces


================
Comment at: clang-tools-extra/docs/ReleaseNotes.rst:207
 
+- Improved :doc:`misc-redundant-expression <clang-tidy/checks/modernize/loop-convert>`
+  to check for container functions ``begin``/``end`` etc on base classes of the container
----------------
Copy-paste typo?




Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D140307



More information about the cfe-commits mailing list