Looks sensible.

> -----Original Message-----
> From: Thomas Wood [mailto:thomas.w...@intel.com]
> Sent: Wednesday, July 23, 2014 11:58 AM
> To: intel-gfx@lists.freedesktop.org
> Cc: Gore, Tim; daniel.vet...@ffwll.ch
> Subject: [PATCH i-g-t 5/8] lib: don't ignore unknown options in multi-tests
> 
> None of the current tests have additional options that might make use of any
> unknown options and igt_subtest_init_parse_opts is available that integrates
> additional option parsing.
> 
> Signed-off-by: Thomas Wood <thomas.w...@intel.com>
Reviewed-by: Tim Gore <tim.g...@intel.com>
> ---
>  lib/igt_core.c | 13 +------------
>  1 file changed, 1 insertion(+), 12 deletions(-)
> 
> diff --git a/lib/igt_core.c b/lib/igt_core.c index 72c77e6..0867c27 100644
> --- a/lib/igt_core.c
> +++ b/lib/igt_core.c
> @@ -459,18 +459,7 @@ enum igt_log_level igt_log_level = IGT_LOG_INFO;
>   */
>  void igt_subtest_init(int argc, char **argv)  {
> -     int ret;
> -
> -     /* supress getopt errors about unknown options */
> -     opterr = 0;
> -
> -     ret = igt_subtest_init_parse_opts(argc, argv, NULL, NULL, NULL,
> NULL);
> -     if (ret < 0)
> -             /* exit with no error for -h/--help */
> -             exit(ret == -1 ? 0 : ret);
> -
> -     /* reset opt parsing */
> -     optind = 1;
> +     igt_subtest_init_parse_opts(argc, argv, NULL, NULL, NULL, NULL);
>  }
> 
>  /**
> --
> 1.9.3

_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/intel-gfx

Reply via email to