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

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 -- Daniel Veillard | libxml Gnome XML XSLT toolkit http://xmlsoft.org/ daniel@veillard.com | Rpmfind RPM search engine http://rpmfind.net/ http://veillard.com/ | virtualization library http://libvirt.org/