On May 7, 2016, at 5:57 AM, Otavio Salvador <[email protected]> wrote: > > From: Tom Hochstein <[email protected]> > > Exit the program if an unrecognized command line option is found. > > Signed-off-by; Tom Hochstein <[email protected]> > Signed-off-by: Otavio Salvador <[email protected]>
Simple enough of a review, and I did test this. But the question is whether we want weston-launch to ignore invalid options or to quit in the event of their presence. I'm not experienced enough to judge, so others will have to chime in. So fwiw, Reviewed-by: Yong Bakos <[email protected]> Tested-by: Yong Bakos <[email protected]> yong > --- > > src/weston-launch.c | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/src/weston-launch.c b/src/weston-launch.c > index b8dfb17..9987d8e 100644 > --- a/src/weston-launch.c > +++ b/src/weston-launch.c > @@ -703,6 +703,8 @@ main(int argc, char *argv[]) > case 'h': > help("weston-launch"); > exit(EXIT_FAILURE); > + default: > + exit(EXIT_FAILURE); > } > } > > -- > 2.8.2 > > _______________________________________________ > wayland-devel mailing list > [email protected] > https://lists.freedesktop.org/mailman/listinfo/wayland-devel _______________________________________________ wayland-devel mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/wayland-devel
