[Libvir] Repository for work-in-progress storage patches

Richard W.M. Jones rjones at redhat.com
Sat Jan 19 13:19:35 UTC 2008


Another thing is that there's confusion over where 'flags' parameters 
should be unsigned or not.  The confusion comes from existing calls 
which are a bit of a hodge-podge, even using 'long' in one case:

virConnectOpenAuth             int flags
virDomainCreateLinux           unsigned int flags
virDomainCreateLinuxJob        unsigned int flags
virDomainReboot                unsigned int flags
virDomainCoreDump              int flags
virDomainCoreDumpJob           int flags
virDomainGetXMLDesc            int flags
virDomainCreateJob             int flags
virDomainMigrate               unsigned long flags
virNetworkGetXMLDesc           int flags
virConnectDiscoverStoragePools unsigned int flags
virStoragePoolRefresh          unsigned int flags
virStoragePoolGetXMLDesc       int flags
virStorageVolCreateXML         int flags
virStorageVolGetXMLDesc        int flags

I propose that we change all except virDomainMigrate to 'unsigned int'. 
  That shouldn't break ABI (right?)

Rich.

-- 
Emerging Technologies, Red Hat - http://et.redhat.com/~rjones/
Registered Address: Red Hat UK Ltd, Amberley Place, 107-111 Peascod
Street, Windsor, Berkshire, SL4 1TE, United Kingdom.  Registered in
England and Wales under Company Registration No. 03798903
-------------- next part --------------
A non-text attachment was scrubbed...
Name: smime.p7s
Type: application/x-pkcs7-signature
Size: 3237 bytes
Desc: S/MIME Cryptographic Signature
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20080119/03cbde8b/attachment-0001.bin>


More information about the libvir-list mailing list