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

[Piotr Padlewski via Phabricator via cfe-commits]

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.


https://reviews.llvm.org/D27806