From: Philippe Mathieu-Daudé Date: Wed, 18 Dec 2019 17:20:06 +0000 (+0100) Subject: monitor/qmp: Explicit we ignore few QEMUChrEvent in IOEventHandler X-Git-Url: http://git.maquefel.me/?a=commitdiff_plain;h=ed7c5bb7ad384f5dd55945db57cae2ba5263a1e4;p=qemu.git monitor/qmp: Explicit we ignore few QEMUChrEvent in IOEventHandler The Chardev events are listed in the QEMUChrEvent enum. To be able to use this enum in the IOEventHandler typedef, we need to explicit all the events ignored by this frontend, to silent the following GCC warning: CC monitor/qmp.o monitor/qmp.c: In function ‘monitor_qmp_event’: monitor/qmp.c:345:5: error: enumeration value ‘CHR_EVENT_BREAK’ not handled in switch [-Werror=switch] 345 | switch (event) { | ^~~~~~ monitor/qmp.c:345:5: error: enumeration value ‘CHR_EVENT_MUX_IN’ not handled in switch [-Werror=switch] monitor/qmp.c:345:5: error: enumeration value ‘CHR_EVENT_MUX_OUT’ not handled in switch [-Werror=switch] cc1: all warnings being treated as errors Signed-off-by: Philippe Mathieu-Daudé Reviewed-by: Marc-André Lureau Reviewed-by: Markus Armbruster Message-Id: <20191218172009.8868-12-philmd@redhat.com> Signed-off-by: Paolo Bonzini --- diff --git a/monitor/qmp.c b/monitor/qmp.c index b67a8e7d1f..6c46be40f8 100644 --- a/monitor/qmp.c +++ b/monitor/qmp.c @@ -365,6 +365,11 @@ static void monitor_qmp_event(void *opaque, int event) mon_refcount--; monitor_fdsets_cleanup(); break; + case CHR_EVENT_BREAK: + case CHR_EVENT_MUX_IN: + case CHR_EVENT_MUX_OUT: + /* Ignore */ + break; } }