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

Malcolm Parsons via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Dec 15 09:25:41 PST 2016

malcolm.parsons added a comment.

What about arguments 3 and 4 of `std::list::splice(It, list, It, It)`?

Comment at: clang-tidy/obvious/ObviousTidyModule.cpp:24
   void addCheckFactories(ClangTidyCheckFactories &CheckFactories) override {
-    // Add obvious checks here.
+    CheckFactories.registerCheck<InvalidRangeCheck>("misc-invalid-range");

Comment at: docs/ReleaseNotes.rst:84
+- New `misc-invalid-range
+  <http://clang.llvm.org/extra/clang-tidy/checks/misc-invalid-range.html>`_ check
Prazek wrote:
> malcolm.parsons wrote:
> > "invalid" < "use"
> You mean to change "invalid" to "use"?
> It is not about using range, it is about finding broken range.
The release notes should be in alphabetical order.


More information about the cfe-commits mailing list