[LTP] [PATCH] lib/tst_test: Warn on repeated options
Jan Stancek
jstancek@redhat.com
Tue Nov 5 14:32:13 CET 2019
----- Original Message -----
> Hi!
> > Signed-off-by: Cyril Hrubis <chrubis@suse.cz>
> > CC: Jan Stancek <jstancek@redhat.com>
> > ---
> > lib/tst_test.c | 3 +++
> > 1 file changed, 3 insertions(+)
> >
> > diff --git a/lib/tst_test.c b/lib/tst_test.c
> > index c7b46d0d5..f41baf3cf 100644
> > --- a/lib/tst_test.c
> > +++ b/lib/tst_test.c
> > @@ -468,6 +468,9 @@ static void parse_topt(unsigned int topts_len, int opt,
> > char *optarg)
> > if (i >= topts_len)
> > tst_brk(TBROK, "Invalid option '%c' (should not happen)", opt);
> >
> > + if (*toptions[i].arg)
> > + tst_res(TWARN, "Option passed -%c multiple times", opt);
>
> Obviously the -%c and passed should be swapped, I can do that before
> pushing if the patch gets ack.
Acked-by: Jan Stancek <jstancek@redhat.com>
More information about the ltp
mailing list