denzor200 added a comment.
My apologize for the delay! Diff was updated, implemented detection of known
failure cases in the matcher and not issue a fixit, just only diagnostic.
================
Comment at: clang-tools-extra/clang-tidy/boost/UseRangeBasedForLoopCheck.cpp:68
+ ColToken->getLocation()),
+ (llvm::Twine("for(") + VariableString + " : ").str());
+ }
----------------
LegalizeAdulthood wrote:
> Now that this is chained, you might need to `clang-format` on it again.
>
> Probably best to just `clang-format` the whole file before you submit.
`clang-format` was already applied
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D116577/new/
https://reviews.llvm.org/D116577
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
