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 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits