Re: [PATCH weston 4/6] shared/option-parser: Add string list options

2013-04-26 Thread Quentin Glidic
On 26/04/2013 11:37, Pekka Paalanen wrote: There's a free(option_modules) further down below, right? What about freeing all the individual strings stored in it? No. But no option (string) is freed while all are strdup()-ed. Otherwise seems ok, though this does change the format of the --mo

Re: [PATCH weston 4/6] shared/option-parser: Add string list options

2013-04-26 Thread Pekka Paalanen
On Tue, 23 Apr 2013 14:54:48 +0200 Quentin Glidic wrote: > From: Quentin Glidic > > Signed-off-by: Quentin Glidic > --- > man/weston.man | 6 +++--- > shared/config-parser.h | 1 + > shared/option-parser.c | 13 + > src/compositor.c | 27 ---

[PATCH weston 4/6] shared/option-parser: Add string list options

2013-04-23 Thread Quentin Glidic
From: Quentin Glidic Signed-off-by: Quentin Glidic --- man/weston.man | 6 +++--- shared/config-parser.h | 1 + shared/option-parser.c | 13 + src/compositor.c | 27 --- tests/weston-tests-env | 6 -- 5 files changed, 45 insertions(+), 8