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.
Christophe