alexfh added a comment. A few style-related comments.
================ Comment at: docs/clang-tidy/index.rst:677 +To check more than one scenario in the same test file use +``-check-suffix=SUFFIX_NAME`` on ``check_clang_tidy.py`` command line. +With ``-check-suffix=SUFFIX_NAME`` you need to replace your ``CHECK-*`` ---------------- s/SUFFIX_NAME/SUFFIX-NAME/ Same below. ================ Comment at: docs/clang-tidy/index.rst:685 + + // RUN: %check_clang_tidy -check-suffix USING-A %s misc-unused-using-decls %t -- -- -DUSING_A + // RUN: %check_clang_tidy -check-suffix USING-B %s misc-unused-using-decls %t -- -- -DUSING_B ---------------- I'd use the ``-check-suffix=...`` format (with the equals sign) for consistency with the documentation. ================ Comment at: test/clang-tidy/check_clang_tidy.py:21 [-assume-filename <file-with-source-extension>] \ + [-check-suffix <FileCheck suffix>] \ <source-file> <check-name> <temp-file> \ ---------------- ``-check-suffix=<file-check-suffix>`` and maybe the same for -assume-filename https://reviews.llvm.org/D45776 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits