On Fri, 2016-10-14 at 15:54 -0400, Laine Stump wrote:
For some reason the values of memballoon model are set using an
anonymous enum, making it impossible to perform nice tricks like
demanding there are cases for all possible values in a switch. This
patch turns the anonymous enum into virDomainMemBalloonModel.
---
src/conf/domain_conf.h | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/src/conf/domain_conf.h b/src/conf/domain_conf.h
index 3e85ae4..d415d2e 100644
--- a/src/conf/domain_conf.h
+++ b/src/conf/domain_conf.h
@@ -1536,13 +1536,13 @@ struct _virDomainRedirFilterDef {
virDomainRedirFilterUSBDevDefPtr *usbdevs;
};
-enum {
+typedef enum {
VIR_DOMAIN_MEMBALLOON_MODEL_VIRTIO,
VIR_DOMAIN_MEMBALLOON_MODEL_XEN,
VIR_DOMAIN_MEMBALLOON_MODEL_NONE,
VIR_DOMAIN_MEMBALLOON_MODEL_LAST
-};
+} virDomainMemBalloonModel;
struct _virDomainMemballoonDef {
int model;
Great catch!
However, you'll want to
s/MemBalloon/Memballoon/
because further down the file you can find
VIR_ENUM_DECL(virDomainMemballoonModel)
I like your name better, by the way, so if you feel like
renameing every instance in a separate patch I'll gladly
review and ACK it :)
ACK with the above taken care of.
--
Andrea Bolognani / Red Hat / Virtualization