On 09/02/2014 03:01 PM, Eric Blake wrote:

>> +        if (!virDomainObjIsActive(vm))
>> +            err = 0;
>> +        else {
> 
> [2] If one leg of if-else has {}, both legs must have it.  This is
> documented in HACKING (and I really ought to add a syntax check that
> forbids obvious cases of unbalanced braces).

As penance for pointing it out, I've written the syntax check:
https://www.redhat.com/archives/libvir-list/2014-September/msg00164.html

-- 
Eric Blake   eblake 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