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@xxxxxxxxxx +1-919-301-3266
Libvirt virtualization library http://libvirt.org
Attachment:
signature.asc
Description: OpenPGP digital signature
-- libvir-list mailing list libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list