On 03/02/2018 10:13 AM, Andrea Bolognani wrote:
Pretty much any reasonable compiler would do this automatically, but there's no harm in being explicit about it.
Signed-off-by: Andrea Bolognani <abologna@redhat.com> --- src/conf/domain_conf.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/conf/domain_conf.h b/src/conf/domain_conf.h index 368f16f3fb..a04f96169c 100644 --- a/src/conf/domain_conf.h +++ b/src/conf/domain_conf.h @@ -700,7 +700,7 @@ typedef enum { } virDomainControllerModelPCI;
typedef enum { - VIR_DOMAIN_CONTROLLER_PCI_MODEL_NAME_NONE, + VIR_DOMAIN_CONTROLLER_PCI_MODEL_NAME_NONE = 0, VIR_DOMAIN_CONTROLLER_PCI_MODEL_NAME_PCI_BRIDGE, VIR_DOMAIN_CONTROLLER_PCI_MODEL_NAME_I82801B11_BRIDGE, VIR_DOMAIN_CONTROLLER_PCI_MODEL_NAME_IOH3420,
I'm not sure why you want it, but don't see any harm (if it's for consistency, there are many other enums that don't explicitly list the first item as 0). Reviewed-by: Laine Stump <laine@laine.org>