hokein added inline comments.
================ Comment at: test/clang-tidy/read_file_config.cpp:5 +// RUN: echo '[{"command": "cc -c -o test.o test.cpp", "directory": "%T/read-file-config", "file": "%T/read-file-config/test.cpp"}]' > %T/read-file-config/compile_commands.json +// RUN: clang-tidy %T/read-file-config/test.cpp | not grep "clang-analyzer-*" + ---------------- alexfh wrote: > Please add a sanity check to ensure the code triggers the static analyzer > check, when enabled explicitly: > > // RUN: clang-tidy -checks=-*,clang-analyzer-* %T/read-file-config/test.cpp | > grep "clang-analyzer-*" > > > (hint: it won't pass, since your regex in grep is incorrect ;) Done. This is valid regex, but doesn't do the expect thing, we expect to match the whole check name, which should be `clang-analyzer-.*` Repository: rCTE Clang Tools Extra https://reviews.llvm.org/D45697 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits