[libvirt] [PATCH 1/3] virDomainCreateXML: support new flag

[Eric Blake]

On 06/08/2010 02:42 PM, Jiri Denemark wrote:
>> diff --git a/include/libvirt/libvirt.h.in b/include/libvirt/libvirt.h.in
>> index 19d5205..c806441 100644
>> --- a/include/libvirt/libvirt.h.in
>> +++ b/include/libvirt/libvirt.h.in
>> @@ -121,7 +121,8 @@ typedef virDomainInfo *virDomainInfoPtr;
>>   * Domain.
>>   */
>>  typedef enum {
>> -     VIR_DOMAIN_NONE = 0
>> +    VIR_DOMAIN_NONE         = 0,      /* Default behavior */
>> +    VIR_DOMAIN_START_PAUSED = 0 << 1, /* Launch guest in paused state */
>>  } virDomainCreateFlags;
> 
> I believe, you wanted to write 1 << 0 as a value of VIR_DOMAIN_START_PAUSED,
> didn't you?

Blame the fact that I was traveling yesterday?  Yes, a bone-headed bug
on my part; thanks for catching it, and thank goodness for the review
process.

> 
> The rest of this patch looks just fine to me, ACK.

Fixed, and will push as soon as I actually complete some testing on 2/3
and 3/3 (which would have prevented posting this bug in the first place).

> 
> Jirka
> 

-- 
Eric Blake   eblake at redhat.com    +1-801-349-2682
Libvirt virtualization library http://libvirt.org

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 619 bytes
Desc: OpenPGP digital signature
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20100608/e075a2ad/attachment-0001.sig>