[PATCH] D27806: [clang-tidy] Add misc-invalid-range

Piotr Padlewski via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Dec 15 09:07:33 PST 2016

Prazek marked an inline comment as done.
Prazek added inline comments.

Comment at: clang-tidy/ClangTidy.cpp:296
   const auto &RegisteredCheckers =
-      AnalyzerOptions::getRegisteredCheckers(/*IncludeExperimental=*/false);
+      AnalyzerOptions::getRegisteredCheckers(/*IncludeExperimental=*/true);
   bool AnalyzerChecksEnabled = false;
malcolm.parsons wrote:
> Is this intentional?
actually not, good catch. I was testing alpha checks

Comment at: docs/clang-tidy/checks/misc-invalid-range.rst:12
+It checks if first and second argument of call is call to ``begin()``
+and ``end()``, but on different objects (having different name).
malcolm.parsons wrote:
> What about the parallel versions of algorithms?
parallel version won't take iterator as first argument, so it won't gonna match.
I don't think it makes sense to support parallel algorithms now because they don't have that many users yet.


More information about the cfe-commits mailing list