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

[Malcolm Parsons via Phabricator via cfe-commits]

malcolm.parsons added a comment.

In https://reviews.llvm.org/D27806#623856, @Prazek wrote:

> So does FIXIT comment satisfies you for this patch?


Yes.



================
Comment at: clang-tidy/obvious/InvalidRangeCheck.cpp:80
+void InvalidRangeCheck::check(const MatchFinder::MatchResult &Result) {
+
+  const auto *FirstArg = Result.Nodes.getNodeAs<DeclRefExpr>("first_arg");
----------------
blank line at start of function.


================
Comment at: docs/clang-tidy/checks/misc-invalid-range.rst:1
+.. title:: clang-tidy - misc-invalid-range
+
----------------
misc.


================
Comment at: test/clang-tidy/misc-invalid-range.cpp:1
+// RUN: %check_clang_tidy %s misc-invalid-range %t
+
----------------
misc


https://reviews.llvm.org/D27806