Re: [PATCH weston v2 1/2] editor: Use parse_options() from shared for command line options

2016-11-21 Thread Bryce Harrington
On Mon, Nov 21, 2016 at 09:08:38PM +0100, Silvan Jegen wrote: > Hi > > > int > > main(int argc, char *argv[]) > > { > > struct editor editor; > > int i; > > This is still unused (as pointed out by Daniel) and should be removed. Oh, thanks, missed that. Bryce > Cheers, > > Silvan >

Re: [PATCH weston v2 1/2] editor: Use parse_options() from shared for command line options

2016-11-21 Thread Silvan Jegen
Hi One comment below. On Mon, Nov 21, 2016 at 10:34:33AM -0800, Bryce Harrington wrote: > Also add a basic --help option > > Signed-off-by: Bryce Harrington > --- > clients/editor.c | 78 > +--- > 1 file changed, 57

[PATCH weston v2 1/2] editor: Use parse_options() from shared for command line options

2016-11-21 Thread Bryce Harrington
Also add a basic --help option Signed-off-by: Bryce Harrington --- clients/editor.c | 78 +--- 1 file changed, 57 insertions(+), 21 deletions(-) diff --git a/clients/editor.c b/clients/editor.c index 30bf555..33b43d2