On Wed, May 29, 2019 at 04:27:33PM -0700, Lucas De Marchi wrote: > We want to check if the long option conflicts with one from the core. > The check for conflicting short option already exists just above. No, this one is checking that the val (the 0) doesn't conflict. -- Petri Latvala > > Signed-off-by: Lucas De Marchi <lucas.demarchi@xxxxxxxxx> > --- > lib/tests/igt_conflicting_args.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/lib/tests/igt_conflicting_args.c b/lib/tests/igt_conflicting_args.c > index c357b6c5..d8be138e 100644 > --- a/lib/tests/igt_conflicting_args.c > +++ b/lib/tests/igt_conflicting_args.c > @@ -91,7 +91,7 @@ int main(int argc, char **argv) > internal_assert_wsignaled(do_fork(), SIGABRT); > > /* conflict on long option 'val' representations */ > - long_options[0] = (struct option) { "iterations", required_argument, NULL, 0}; > + long_options[0] = (struct option) { "list-subtests", required_argument, NULL, 0}; > short_options = ""; > internal_assert_wsignaled(do_fork(), SIGABRT); > > -- > 2.21.0 > > _______________________________________________ > Intel-gfx mailing list > Intel-gfx@xxxxxxxxxxxxxxxxxxxxx > https://lists.freedesktop.org/mailman/listinfo/intel-gfx _______________________________________________ Intel-gfx mailing list Intel-gfx@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/intel-gfx