'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@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list