Re: [PATCH i-g-t 5/8] lib: don't ignore unknown options in multi-tests

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Looks sensible.

> -----Original Message-----
> From: Thomas Wood [mailto:thomas.wood@xxxxxxxxx]
> Sent: Wednesday, July 23, 2014 11:58 AM
> To: intel-gfx@xxxxxxxxxxxxxxxxxxxxx
> Cc: Gore, Tim; daniel.vetter@xxxxxxxx
> 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.wood@xxxxxxxxx>
Reviewed-by: Tim Gore <tim.gore@xxxxxxxxx>
> ---
>  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@xxxxxxxxxxxxxxxxxxxxx
http://lists.freedesktop.org/mailman/listinfo/intel-gfx




[Index of Archives]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]
  Powered by Linux