labath added a comment.

Seems reasonable, just make the option name consistent.



================
Comment at: packages/Python/lldbsuite/test/dotest_args.py:85-88
         dest='skip_categories',
         help=textwrap.dedent('''Specify categories of test cases to skip. 
Takes precedence over -G. Can be specified more than once.'''))
+    group.add_argument(
+        '--xfail-category',
----------------
Please use a consistent name for the two options.


Repository:
  rLLDB LLDB

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D71906/new/

https://reviews.llvm.org/D71906



_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to