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   ebl...@redhat.com    +1-919-301-3266
Libvirt virtualization library http://libvirt.org

Attachment: signature.asc
Description: OpenPGP digital signature

--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list

Reply via email to