[PATCH] D116577: [clang-tidy] Added "boost-use-range-based-for-loop" check

[Denis Mikhailov via Phabricator via cfe-commits]

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