lebedev.ri added a comment.

Thank you!



================
Comment at: test/clang-tidy/check_clang_tidy.py:94-95
+    for check in args.check_suffix:
+      if not re.match('^[A-Z0-9\-,]+$', check):
+        sys.exit('Only A..Z, 0..9, "," and "-" are ' +
+          'allowed in check suffixes list, but "%s" was given' % (check))
----------------
How `,` can be allowed if that is the suffix separator?
Also, i think `_` can be supported.


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

Reply via email to