Re: [E-devel] some bug fix patches

2011-01-31 Thread The Rasterman
] some bug fix patches Right. It looks wrong to me too. Spank Hermet! Daniel Juyung Seo (SeoZ) On Fri, Jan 21, 2011 at 10:04 AM, Lucas De Marchi lucas.demar...@profusion.mobi wrote: Hi Hermet On Wed, Jan 19, 2011 at 9:32 AM, ChunEon Park chuneon.p...@samsung.com wrote: Hi

Re: [E-devel] some bug fix patches

2011-01-20 Thread Daniel Juyung Seo
Right. It looks wrong to me too. Spank Hermet! Daniel Juyung Seo (SeoZ) On Fri, Jan 21, 2011 at 10:04 AM, Lucas De Marchi lucas.demar...@profusion.mobi wrote: Hi Hermet On Wed, Jan 19, 2011 at 9:32 AM, ChunEon Park chuneon.p...@samsung.com wrote: Hi, This is Hermet. Here are simple

Re: [E-devel] some bug fix patches

2011-01-20 Thread Mike McCormack
On 01/21/2011 10:04 AM, Lucas De Marchi wrote: Patch 001_evas.patch.txt does not look right. Besides the trailing whitespaces, it doesn't make sense. Maybe you forgot to save your changes before sending? -free(g); -free(g-keyname); I think the essence of the patch is

Re: [E-devel] some bug fix patches

2011-01-20 Thread Tom Hacohen
On Fri, 2011-01-21 at 13:23 +0900, Mike McCormack wrote: - free(g); - free(g-keyname); I think the essence of the patch is that the above two lines are in the wrong order. It would be clearer if whitespace changes were not mixed with fixes. Actually, I didn't even

Re: [E-devel] some bug fix patches

2011-01-20 Thread ChunEon Park
: Daniel Juyung Seo [mailto:seojuyu...@gmail.com] Sent: Friday, January 21, 2011 11:41 AM To: Lucas De Marchi Cc: ChunEon Park; enlightenment-devel@lists.sourceforge.net Subject: Re: [E-devel] some bug fix patches Right. It looks wrong to me too. Spank Hermet! Daniel Juyung Seo (SeoZ) On Fri, Jan 21

[E-devel] some bug fix patches

2011-01-19 Thread ChunEon Park
Hi, This is Hermet. Here are simple patches for invalid pointers Please review then apply them. Danke! Index: src/lib/eet_lib.c === --- src/lib/eet_lib.c (revision 56208) +++ src/lib/eet_lib.c (working copy) @@ -1725,11