Re: [PATCH weston v4 5/5] Switch to use safe_strtoint instead of strtol

2016-11-16 Thread Daniel Stone
Hi, On 10 August 2016 at 09:31, Pekka Paalanen wrote: > On Tue, 9 Aug 2016 11:12:33 -0700 > Bryce Harrington wrote: >> On Mon, Aug 08, 2016 at 02:55:14PM +0300, Pekka Paalanen wrote: >> > there's a new warning: >> > >> > /home/pq/git/weston/compositor/systemd-notify.c: In function ‘module_init’:

Re: [PATCH weston v4 5/5] Switch to use safe_strtoint instead of strtol

2016-08-10 Thread Pekka Paalanen
On Tue, 9 Aug 2016 11:12:33 -0700 Bryce Harrington wrote: > On Mon, Aug 08, 2016 at 02:55:14PM +0300, Pekka Paalanen wrote: > > On Wed, 3 Aug 2016 17:40:52 -0700 > > Bryce Harrington wrote: > > > > > Reviewed-by: Eric Engestrom > > > Signed-off-by: Bryce Harrington > > > --- > > > composi

Re: [PATCH weston v4 5/5] Switch to use safe_strtoint instead of strtol

2016-08-09 Thread Bryce Harrington
On Mon, Aug 08, 2016 at 02:55:14PM +0300, Pekka Paalanen wrote: > On Wed, 3 Aug 2016 17:40:52 -0700 > Bryce Harrington wrote: > > > Reviewed-by: Eric Engestrom > > Signed-off-by: Bryce Harrington > > --- > > compositor/main.c | 7 +++ > > compositor/systemd-notify.c | 9 +++---

Re: [PATCH weston v4 5/5] Switch to use safe_strtoint instead of strtol

2016-08-08 Thread Pekka Paalanen
On Wed, 3 Aug 2016 17:40:52 -0700 Bryce Harrington wrote: > Reviewed-by: Eric Engestrom > Signed-off-by: Bryce Harrington > --- > compositor/main.c | 7 +++ > compositor/systemd-notify.c | 9 +++-- > libweston/compositor.c | 9 +++-- > libweston/libbacklight.c

Re: [PATCH weston v4 5/5] Switch to use safe_strtoint instead of strtol

2016-08-06 Thread Bryce Harrington
On Fri, Aug 05, 2016 at 08:43:30AM +1000, Peter Hutterer wrote: > On Wed, Aug 03, 2016 at 05:40:52PM -0700, Bryce Harrington wrote: > > Reviewed-by: Eric Engestrom > > Signed-off-by: Bryce Harrington > > series: Reviewed-by: Peter Hutterer Thanks, pushed: 295e9d0..25a2bdd master -> master

Re: [PATCH weston v4 5/5] Switch to use safe_strtoint instead of strtol

2016-08-04 Thread Peter Hutterer
On Wed, Aug 03, 2016 at 05:40:52PM -0700, Bryce Harrington wrote: > Reviewed-by: Eric Engestrom > Signed-off-by: Bryce Harrington series: Reviewed-by: Peter Hutterer Cheers, Peter > --- > compositor/main.c | 7 +++ > compositor/systemd-notify.c | 9 +++-- > libweston/co

[PATCH weston v4 5/5] Switch to use safe_strtoint instead of strtol

2016-08-03 Thread Bryce Harrington
Reviewed-by: Eric Engestrom Signed-off-by: Bryce Harrington --- compositor/main.c | 7 +++ compositor/systemd-notify.c | 9 +++-- libweston/compositor.c | 9 +++-- libweston/libbacklight.c| 11 +-- shared/config-parser.c | 7 ++- shared/option-