[libvirt] [PATCH 02/48] Destroy virdomainlist.[ch]

Eric Blake eblake at redhat.com
Tue Aug 14 11:45:24 UTC 2012


On 08/14/2012 03:28 AM, Osier Yang wrote:

>>           for (i = 0; i<  count; i++) {
>>               if (data.domains[i])
>> -                virUnrefDomain(data.domains[i]);
> 
> Not sure if it will cause conflicts to you, but the "if" is removed
> for "avoid_if_before_free".

Nah, 'make syntax-check' made me realize the same thing, after I had
posted my review.  Thanks for pushing, I'll start posting my patches
after another round of polish on my end.

-- 
Eric Blake   eblake at redhat.com    +1-919-301-3266
Libvirt virtualization library http://libvirt.org

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 620 bytes
Desc: OpenPGP digital signature
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20120814/42b66b20/attachment-0001.sig>


More information about the libvir-list mailing list