Re: [PATCH libvirt-glib] gobject: add GVir.DomainShutdownFlags binding

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

 



On Fri, Aug 3, 2012 at 12:54 PM, Christophe Fergeau <cfergeau@xxxxxxxxxx> wrote:
> On Fri, Aug 03, 2012 at 12:44:10PM +0200, Marc-André Lureau wrote:
>> The counterpart GVirDomainStartFlags uses _NONE. Is this a "flawed" in
>> libvirt API? Do we prefer to copy libvirt or do we want consitancy? I
>> prefer constitancy.
>
> I don't think this is a flaw, it's just that '0' has a different meaning
> depending on the API. Sometimes 0 means 'no flag', ie no additional
> behaviour, in this 'default' case, it means that the behaviour will be
> equivalent to one of the flags, but we don't indicate which one, we let the
> hypervisor choose the default one.
> So having different names for the enum for the 0 value doesn't strike me as
> inconsistent as they indicate slightly different things.

Yes, but in this particual case :

virDomainShutdownFlagValues VIR_DOMAIN_SHUTDOWN_DEFAULT        = 0,
    /* hypervisor choice */

(in libvirt Create means Launch or Start)

virDomainCreateFlags VIR_DOMAIN_NONE               = 0,      /*
Default behavior */

It would make sense if libvirt had VIR_DOMAIN_START_DEFAULT instead
(the rest of the values have DOMAIN_START prefix too..), and we use
_DEFAULT too for something that leaves libvirt hypervisor backend
choose the behavior.


-- 
Marc-André Lureau

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



[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]