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(a)redhat.com +1-919-301-3266
Libvirt virtualization library http://libvirt.org
Attachments:
- signature.asc (application/pgp-signature — 620 bytes)