[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]

Re: [libvirt] [PATCH] Add virBufferFreeAndReset() and replace free()



2009/12/9 Daniel Veillard <veillard redhat com>:
> On Sun, Nov 22, 2009 at 07:22:08PM +0100, Matthias Bolte wrote:
>> Replace free(virBufferContentAndReset()) with virBufferFreeAndReset().
>> Update documentation and replace all remaining calls to free() with
>> calls to VIR_FREE(). Also add missing calls to virBufferFreeAndReset()
>> and virReportOOMError() in OOM error cases.
>
>  Looks fine, nice cleanup, fix a few potential leaks too  and a few
> indentation mistakes, ACK
>
> Daniel
>

Just converted some remaining free()'s to VIR_FREE() that I've missed last time.

Okay, pushed.

Matthias


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]