[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 =
bool AnalyzerChecksEnabled = false;
> 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).
> 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