Re: [PATCH] clients: Free resources on exit in eventdemo

2012-07-09 Thread Pekka Paalanen
On Sun, 08 Jul 2012 15:59:44 +0200 Martin Olsson wrote: > On 07/08/2012 03:09 PM, Pekka Paalanen wrote: > > Looks good to me now, I assume you tested this program exits > > properly. Btw. have you tried Valgrind on these? > > Yes I tried it; FWIW with the new call to display_destroy the > event

Re: [PATCH] clients: Free resources on exit in eventdemo

2012-07-08 Thread Martin Olsson
On 07/08/2012 03:09 PM, Pekka Paalanen wrote: Looks good to me now, I assume you tested this program exits properly. Btw. have you tried Valgrind on these? Yes I tried it; FWIW with the new call to display_destroy the eventdemo app now hits invalid read, invalid write and finally an assert. H

Re: [PATCH] clients: Free resources on exit in eventdemo

2012-07-08 Thread Pekka Paalanen
On Sun, 8 Jul 2012 11:11:18 +0200 Martin Olsson wrote: > --- > clients/eventdemo.c | 15 +++ > 1 file changed, 15 insertions(+) > > diff --git a/clients/eventdemo.c b/clients/eventdemo.c > index daf3283..22dade3 100644 > --- a/clients/eventdemo.c > +++ b/clients/eventdemo.c > @@

[PATCH] clients: Free resources on exit in eventdemo

2012-07-08 Thread Martin Olsson
--- clients/eventdemo.c | 15 +++ 1 file changed, 15 insertions(+) diff --git a/clients/eventdemo.c b/clients/eventdemo.c index daf3283..22dade3 100644 --- a/clients/eventdemo.c +++ b/clients/eventdemo.c @@ -261,6 +261,7 @@ motion_handler(struct widget *widget, struct input *input,