JonasToth added a comment. The change looks good in principle. I think it would make sense to migrate one test already, to use the new capability and check if everything works as expected. The current tests still run fine?
================ Comment at: test/clang-tidy/check_clang_tidy.py:112 + sys.exit('%s, %s or %s not found in the input' % + (check_fixes_prefixes, check_messages_prefixes, check_notes_prefixes)) ---------------- i think the whitespace in this line should be aligned with with the line above Repository: rCTE Clang Tools Extra https://reviews.llvm.org/D52971 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits