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

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



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

Attachment: signature.asc
Description: OpenPGP digital signature


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