Re: [libvirt] [PATCHv2 21/27] node_device: avoid implicit int

2011-07-13 Thread Matthias Bolte
2011/7/8 Eric Blake :
> 'unsigned a' and 'unsigned int a' are synonyms, but we generally
> always spell out the 'int' in that case.  Fixing this will avoid
> a false positive in the next syntax-check commit.
>
> * src/conf/node_device_conf.h (pci_config_address)
> (_virNodeDevCapsDef): Prefer 'unsigned int' over 'unsigned'.
> ---
>
> v2: new patch.
> I didn't add a syntax check for implicit int, though, as we have a
> lot of other cases of that.  Rather, this was the only hit when
> grepping for 'unsigned flags'.
>
>  src/conf/node_device_conf.h |   58 +-
>  1 files changed, 29 insertions(+), 29 deletions(-)

ACK.

-- 
Matthias Bolte
http://photron.blogspot.com

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

[libvirt] [PATCHv2 21/27] node_device: avoid implicit int

2011-07-08 Thread Eric Blake
'unsigned a' and 'unsigned int a' are synonyms, but we generally
always spell out the 'int' in that case.  Fixing this will avoid
a false positive in the next syntax-check commit.

* src/conf/node_device_conf.h (pci_config_address)
(_virNodeDevCapsDef): Prefer 'unsigned int' over 'unsigned'.
---

v2: new patch.
I didn't add a syntax check for implicit int, though, as we have a
lot of other cases of that.  Rather, this was the only hit when
grepping for 'unsigned flags'.

 src/conf/node_device_conf.h |   58 +-
 1 files changed, 29 insertions(+), 29 deletions(-)

diff --git a/src/conf/node_device_conf.h b/src/conf/node_device_conf.h
index e90bdc5..cef86d4 100644
--- a/src/conf/node_device_conf.h
+++ b/src/conf/node_device_conf.h
@@ -83,10 +83,10 @@ enum virNodeDevPCICapFlags {
 };

 struct pci_config_address {
-unsigned domain;
-unsigned bus;
-unsigned slot;
-unsigned function;
+unsigned int domain;
+unsigned int bus;
+unsigned int slot;
+unsigned int function;
 };

 typedef struct _virNodeDevCapsDef virNodeDevCapsDef;
@@ -109,55 +109,55 @@ struct _virNodeDevCapsDef {
 } firmware;
 } system;
 struct {
-unsigned domain;
-unsigned bus;
-unsigned slot;
-unsigned function;
-unsigned product;
-unsigned vendor;
-unsigned class;
+unsigned int domain;
+unsigned int bus;
+unsigned int slot;
+unsigned int function;
+unsigned int product;
+unsigned int vendor;
+unsigned int class;
 char *product_name;
 char *vendor_name;
 struct pci_config_address *physical_function;
 struct pci_config_address **virtual_functions;
-unsigned num_virtual_functions;
-unsigned flags;
+unsigned int num_virtual_functions;
+unsigned int flags;
 } pci_dev;
 struct {
-unsigned bus;
-unsigned device;
-unsigned product;
-unsigned vendor;
+unsigned int bus;
+unsigned int device;
+unsigned int product;
+unsigned int vendor;
 char *product_name;
 char *vendor_name;
 } usb_dev;
 struct {
-unsigned number;
-unsigned _class;   /* "class" is reserved in C */
-unsigned subclass;
-unsigned protocol;
+unsigned int number;
+unsigned int _class;   /* "class" is reserved in C */
+unsigned int subclass;
+unsigned int protocol;
 char *description;
 } usb_if;
 struct {
 char *address;
-unsigned address_len;
+unsigned int address_len;
 char *ifname;
 enum virNodeDevNetCapType subtype;  /* LAST -> no subtype */
 } net;
 struct {
-unsigned host;
+unsigned int host;
 char *wwnn;
 char *wwpn;
-unsigned flags;
+unsigned int flags;
 } scsi_host;
 struct {
 char *name;
 } scsi_target;
 struct {
-unsigned host;
-unsigned bus;
-unsigned target;
-unsigned lun;
+unsigned int host;
+unsigned int bus;
+unsigned int target;
+unsigned int lun;
 char *type;
 } scsi;
 struct {
@@ -172,7 +172,7 @@ struct _virNodeDevCapsDef {
 char *vendor;
 char *serial;
 char *media_label;
-unsigned flags;/* virNodeDevStorageCapFlags bits */
+unsigned int flags;/* virNodeDevStorageCapFlags bits */
 } storage;
 } data;
 virNodeDevCapsDefPtr next;  /* next capability */
-- 
1.7.4.4

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