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

Thu Jan 5 17:26:34 PST 2023

ccotter marked an inline comment as done.
ccotter 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
----------------
ccotter wrote:
> carlosgalvezp wrote:
> > Replace tabs with spaces
> Ah, I think this was before I figured out `arc diff` and was copy/pasting diff files around :/
> 
> Thanks for catching this.
just to confirm, how did you notice the tabs? I couldnt find them when I downloaded the phab raw diff for any of the versions in the history. I still see a double right arrow `»` in the phab diff - is that indicating a tab or something else?

As a sanity check, I ran `./clang/tools/clang-format/git-clang-format --binary build/bin/clang-format` from the root of my project and the tool did not produce any changes with the latest version of the changes I submitted.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D140307