njames93 marked 3 inline comments as done. njames93 added inline comments.
================ Comment at: clang-tools-extra/clang-tidy/modernize/LoopConvertCheck.cpp:859 const auto *LoopVar = Nodes.getNodeAs<VarDecl>(InitVarName); - const auto *EndVar = Nodes.getNodeAs<VarDecl>(EndVarName); - - // If the loop calls end()/size() after each iteration, lower our confidence - // level. - if (FixerKind != LFK_Array && !EndVar) - ConfidenceLevel.lowerTo(Confidence::CL_Reasonable); + const VarDecl *EndVar = Nodes.getNodeAs<VarDecl>(EndVarName); ---------------- steveire wrote: > Why change `auto` to `VarDecl` (repeated) here? Changed it when I removed the initializer, forget to undo it when I added the initializer back. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D97955/new/ https://reviews.llvm.org/D97955 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
