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

denzor200 planned changes to this revision. denzor200 added inline comments. Comment at: clang-tools-extra/test/clang-tidy/checkers/boost-use-range-based-for-loop.cpp:22 + +#define BOOST_FOREACH(VAR, COL) for(VAR;(COL, false);) +#define IDENTITY_TYPE(...) __VA_ARGS__ ---

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

denzor200 updated this revision to Diff 411556. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D116577/new/ https://reviews.llvm.org/D116577 Files: clang-tools-extra/clang-tidy/boost/BoostTidyModule.cpp clang-tools-extra/clang-tidy/boost/CMakeLists.txt clang-tools-extra/clang-tidy/bo

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

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 +

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

denzor200 updated this revision to Diff 411548. denzor200 marked 6 inline comments as done. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D116577/new/ https://reviews.llvm.org/D116577 Files: clang-tools-extra/clang-tidy/boost/BoostTidyModule.cpp clang-tools-extra/clang-tidy/boost/CMak

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

denzor200 marked an inline comment as done. denzor200 added inline comments. Comment at: clang-tools-extra/clang-tidy/boost/CMakeLists.txt:8 BoostTidyModule.cpp + UseRangeBasedForLoopCheck.cpp UseToStringCheck.cpp LegalizeAdulthood wrote: > I am wondering

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

denzor200 updated this revision to Diff 398517. denzor200 added a comment. clang-format Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D116577/new/ https://reviews.llvm.org/D116577 Files: clang-tools-extra/clang-tidy/boost/BoostTidyModule.cpp cl

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

denzor200 updated this revision to Diff 398516. denzor200 marked 9 inline comments as done. denzor200 added a comment. review Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D116577/new/ https://reviews.llvm.org/D116577 Files: clang-tools-extra/cla

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

denzor200 added a comment. In D116577#3220757 , @LegalizeAdulthood wrote: > I opened a similar issue for converting Qt's foreach to a range for loop > . > > However this check lands, it should be a simp

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

denzor200 created this revision. Herald added subscribers: carlosgalvezp, xazax.hun, mgorny. denzor200 requested review of this revision. Herald added a project: clang-tools-extra. Added a check for converting ``BOOST_FOREACH(..., ...)`` loops to use the new range-based loops in C++11. This check