On 05/03/14 14:45, Gustavo Sverzut Barbieri wrote:
>> - if (num > 0)
>> + if ((str) && (str[0]))
>> {
>> free(str[0]);
>> free(str);
>
> you could do just if (str) { free(str[0]); free(str); }
>
> free(str[0]) if str[0] is NULL is okay.
>
> I'm not sure if it would return str in that case, if it did it would
> leak with your check.
>
>
Sure. Could have been done that way. It's done This way in other code
with similar usage, so I just kept with that.
dh
------------------------------------------------------------------------------
Subversion Kills Productivity. Get off Subversion & Make the Move to Perforce.
With Perforce, you get hassle-free workflows. Merge that actually works.
Faster operations. Version large binaries. Built-in WAN optimization and the
freedom to use Git, Perforce or both. Make the move to Perforce.
http://pubads.g.doubleclick.net/gampad/clk?id=122218951&iu=/4140/ostg.clktrk
_______________________________________________
enlightenment-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/enlightenment-devel
