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");
   }
----------------
misc?


================
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.


https://reviews.llvm.org/D27806



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to