<div dir="ltr">Looks fine, in the future it seems like a good cleanup would be to move all the argument validation into dotest_args, but this seems fine for now.<br></div><br><div class="gmail_quote">On Fri, May 22, 2015 at 7:54 AM Vince Harron <<a href="mailto:vince@nethacker.com">vince@nethacker.com</a>> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi clayborg, zturner,<br>
<br>
<a href="https://urldefense.proofpoint.com/v2/url?u=http-3A__reviews.llvm.org_D9937&d=AwMFaQ&c=8hUWFZcy2Z-Za5rBPlktOQ&r=MEqT8U_n7oNfuDW5NRbY3ZV384ZquXIYFPWmprwUdKM&m=lXoGEncx34I1uEL0jcvWEI7OmK8LO9rEYUdTFQR1cGc&s=PqEaYm34RjuZQxwZmlwYLTPtPWszmsVwR28ZJ4cbpNc&e=" target="_blank">http://reviews.llvm.org/D9937</a><br>
<br>
Files:<br>
  test/dosep.py<br>
  test/dotest.py<br>
  test/dotest_args.py<br>
<br>
Index: test/dosep.py<br>
===================================================================<br>
--- test/dosep.py<br>
+++ test/dosep.py<br>
@@ -233,7 +233,10 @@<br>
<br>
     is_posix = (<a href="https://urldefense.proofpoint.com/v2/url?u=http-3A__os.name&d=AwMFaQ&c=8hUWFZcy2Z-Za5rBPlktOQ&r=MEqT8U_n7oNfuDW5NRbY3ZV384ZquXIYFPWmprwUdKM&m=lXoGEncx34I1uEL0jcvWEI7OmK8LO9rEYUdTFQR1cGc&s=DRwv2wnv6U7bt6rY4gXXwaQ4iAiM6RzqwUh55SUFygw&e=" target="_blank">os.name</a> == "posix")<br>
     dotest_argv = shlex.split(dotest_option_string, posix=is_posix) if dotest_option_string else []<br>
-    dotest_options = dotest_args.getArguments(dotest_argv)<br>
+<br>
+    parser = dotest_args.create_parser()<br>
+    dotest_options = dotest_args.parse_args(parser, dotest_argv)<br>
+<br>
     if not dotest_options.s:<br>
         # no session log directory, we need to add this to prevent<br>
         # every dotest invocation from creating its own directory<br>
Index: test/dotest.py<br>
===================================================================<br>
--- test/dotest.py<br>
+++ test/dotest.py<br>
@@ -491,7 +491,8 @@<br>
     platform_system = platform.system()<br>
     platform_machine = platform.machine()<br>
<br>
-    args = dotest_args.getArguments(sys.argv[1:])<br>
+    parser = dotest_args.create_parser()<br>
+    args = dotest_args.parse_args(parser, sys.argv[1:])<br>
<br>
     if args.unset_env_varnames:<br>
         for env_var in args.unset_env_varnames:<br>
Index: test/dotest_args.py<br>
===================================================================<br>
--- test/dotest_args.py<br>
+++ test/dotest_args.py<br>
@@ -26,8 +26,7 @@<br>
     else:<br>
         return parser.parse_args(args=argv)<br>
<br>
-def getArguments(argv):<br>
-<br>
+def create_parser():<br>
     parser = argparse.ArgumentParser(description='description', prefix_chars='+-', add_help=False)<br>
     group = None<br>
<br>
@@ -114,4 +113,4 @@<br>
     group = parser.add_argument_group('Test directories')<br>
     group.add_argument('args', metavar='test-dir', nargs='*', help='Specify a list of directory names to search for test modules named after Test*.py (test discovery). If empty, search from the current working directory instead.')<br>
<br>
-    return parse_args(parser, argv)<br>
+    return parser<br>
<br>
EMAIL PREFERENCES<br>
  <a href="https://urldefense.proofpoint.com/v2/url?u=http-3A__reviews.llvm.org_settings_panel_emailpreferences_&d=AwMFaQ&c=8hUWFZcy2Z-Za5rBPlktOQ&r=MEqT8U_n7oNfuDW5NRbY3ZV384ZquXIYFPWmprwUdKM&m=lXoGEncx34I1uEL0jcvWEI7OmK8LO9rEYUdTFQR1cGc&s=mjFS295tc3ktZo48BoTsvJ2tjyFTh54JhGlH1RGiB9g&e=" target="_blank">http://reviews.llvm.org/settings/panel/emailpreferences/</a><br>
</blockquote></div>