[Freeassociation-devel] Memory leak patches 1 of many

Alvaro Manera alvaro.manera at nokia.com
Sun May 31 23:24:38 PDT 2009


Hi,

Yes, you are right... So here are again fixed (and not sending some unneeded 
as before sorry ;))

And I will start sending more soon...

Alvaro

On Friday 29 May 2009 19:32:41 ext dothebart wrote:
> Hy alvaro, looking good for me except one conceptional thing...
>
> In order to stay clean C also on older compilers or warningless in more
> strict modes of gcc, could you please not create code before declaration?
>
> so please make it like that:
>
> declaration struct;
>
> pointer_check
>
> struct = malloc()
>
>
>
> if you add  -pedantic to gcc's CFLAGS it will warn you about it.
>
> for the rest it looks ok at first glance.
>
>
>
> Wilfried Goesgens


-------------- next part --------------
A non-text attachment was scrubbed...
Name: autogen.diff
Type: text/x-diff
Size: 1186 bytes
Desc: not available
URL: <http://lists.infradead.org/pipermail/libical-devel/attachments/20090601/26904fdd/attachment.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: icaltypes_error_order.diff
Type: text/x-diff
Size: 506 bytes
Desc: not available
URL: <http://lists.infradead.org/pipermail/libical-devel/attachments/20090601/26904fdd/attachment-0001.bin>


More information about the libical-devel mailing list