On Fri, Nov 13, 2009 at 1:15 AM, Xavier <[email protected]> wrote:
> On Fri, Nov 13, 2009 at 12:59 AM, Laszlo Papp <[email protected]> wrote:
>> * It makes the code clearer to read/understand
>> * Cppcheck tool doesn't show this anymore: [./util.c:215]: (error) Resource 
>> leak: fd
>> ---
>>  lib/libalpm/trans.c |    5 ++++-
>>  lib/libalpm/util.c  |   11 +++++++----
>>  src/pacman/pacman.c |    4 +++-
>>  3 files changed, 14 insertions(+), 6 deletions(-)
>>
>
> The 6th iteration of this trivial patch looks good to me.
>
> Thanks for your hard work :)
>
>

Okay, Dan applied it, thank you the helps Xavier, really.

Best Regards,
Laszlo Papp

Reply via email to