A few more comments on the test.
================ Comment at: test/clang-tidy/llvm-arg-comments.cpp:1 @@ +1,2 @@ +// RUN: clang-tidy --checks=llvm-argument-comment %s -- > %t.cpp +// RUN: FileCheck -input-file=%t.cpp %s ---------------- You could also avoid using a temporary file and just pipe the clang-tidy output to FileCheck. ================ Comment at: test/clang-tidy/llvm-arg-comments.cpp:3 @@ +2,3 @@ +// RUN: FileCheck -input-file=%t.cpp %s + +void f(int x, int y); ---------------- Please add // CHECK-NOT: warning here and in the end of the file. I didn't find a better way to check for the absence of the unexpected diagnostics. Maybe we should implement -verify in clang-tidy as it's done in Clang. http://llvm-reviews.chandlerc.com/D2914 _______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
