[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [RFC PATCH 10/14] monitor/hmp: Explicit we ignore a QEMUChrEvent in
From: |
Dr. David Alan Gilbert |
Subject: |
Re: [RFC PATCH 10/14] monitor/hmp: Explicit we ignore a QEMUChrEvent in IOEventHandler |
Date: |
Tue, 17 Dec 2019 17:51:39 +0000 |
User-agent: |
Mutt/1.13.0 (2019-11-30) |
* Philippe Mathieu-Daudé (address@hidden) wrote:
> On 12/17/19 6:37 PM, Dr. David Alan Gilbert wrote:
> > * Philippe Mathieu-Daudé (address@hidden) wrote:
> > > The Chardev events are listed in the QEMUChrEvent enum. To be
> > > able to use this enum in the IOEventHandler typedef, we need to
> > > explicit when frontends ignore some events, to silent GCC the
> > > following warnings:
> > >
> > > CC monitor/hmp.o
> > > monitor/hmp.c: In function ‘monitor_event’:
> > > monitor/hmp.c:1330:5: error: enumeration value ‘CHR_EVENT_BREAK’ not
> > > handled in switch [-Werror=switch]
> > > 1330 | switch (event) {
> > > | ^~~~~~
> > > cc1: all warnings being treated as errors
> > >
> > > Signed-off-by: Philippe Mathieu-Daudé <address@hidden>
> > > ---
> >
> >
> > Acked-by: Dr. David Alan Gilbert <address@hidden>
> > for HMP
> >
> > Note that the use of 'default' will make life more unpredictable
> > if you ever come to add a new event type.
>
> You are right, this patch not good as it dumbly ignore the warning...
> I will add all the missing cases:
>
> -- >8 --
> @@ -1371,6 +1371,10 @@ static void monitor_event(void *opaque, int event)
> mon_refcount--;
> monitor_fdsets_cleanup();
> break;
> +
> + case CHR_EVENT_BREAK:
> + /* Ignored */
> + break;
> }
> }
>
> ---
>
> And keep your Acked-by. Thanks!
Acked-by: Dr. David Alan Gilbert <address@hidden>
>
> > > Cc: "Dr. David Alan Gilbert" <address@hidden>
> > > ---
> > > monitor/hmp.c | 4 ++++
> > > 1 file changed, 4 insertions(+)
> > >
> > > diff --git a/monitor/hmp.c b/monitor/hmp.c
> > > index 8942e28933..d84238c120 100644
> > > --- a/monitor/hmp.c
> > > +++ b/monitor/hmp.c
> > > @@ -1371,6 +1371,10 @@ static void monitor_event(void *opaque, int event)
> > > mon_refcount--;
> > > monitor_fdsets_cleanup();
> > > break;
> > > +
> > > + default:
> > > + /* Ignore */
> > > + break;
> > > }
> > > }
> > > --
> > > 2.21.0
> > >
> > --
> > Dr. David Alan Gilbert / address@hidden / Manchester, UK
> >
>
--
Dr. David Alan Gilbert / address@hidden / Manchester, UK
- [RFC PATCH 03/14] vhost-user: Explicit we ignore some QEMUChrEvent in IOEventHandler, (continued)
- [RFC PATCH 03/14] vhost-user: Explicit we ignore some QEMUChrEvent in IOEventHandler, Philippe Mathieu-Daudé, 2019/12/17
- [RFC PATCH 04/14] virtio-console: Explicit we ignore some QEMUChrEvent in IOEventHandler, Philippe Mathieu-Daudé, 2019/12/17
- [RFC PATCH 05/14] hw/ipmi: Explicit we ignore some QEMUChrEvent in IOEventHandler, Philippe Mathieu-Daudé, 2019/12/17
- [RFC PATCH 06/14] hw/usb/dev-serial: Explicit we ignore few QEMUChrEvent in IOEventHandler, Philippe Mathieu-Daudé, 2019/12/17
- [RFC PATCH 07/14] ccid-card-passthru: Explicit we ignore QEMUChrEvent in IOEventHandler, Philippe Mathieu-Daudé, 2019/12/17
- [RFC PATCH 08/14] hw/usb/redirect: Explicit we ignore few QEMUChrEvent in IOEventHandler, Philippe Mathieu-Daudé, 2019/12/17
- [RFC PATCH 09/14] monitor/qmp: Explicit we ignore few QEMUChrEvent in IOEventHandler, Philippe Mathieu-Daudé, 2019/12/17
- [RFC PATCH 10/14] monitor/hmp: Explicit we ignore a QEMUChrEvent in IOEventHandler, Philippe Mathieu-Daudé, 2019/12/17
- [RFC PATCH 11/14] net/vhost-user: Explicit we ignore few QEMUChrEvent in IOEventHandler, Philippe Mathieu-Daudé, 2019/12/17
- [RFC PATCH 12/14] vhost-user-blk: Explicit we ignore few QEMUChrEvent in IOEventHandler, Philippe Mathieu-Daudé, 2019/12/17
- [RFC PATCH 13/14] hw/char/terminal3270: Explicit ignored QEMUChrEvent in IOEventHandler, Philippe Mathieu-Daudé, 2019/12/17
- [PATCH 14/14] chardev: Use QEMUChrEvent enum in IOEventHandler typedef, Philippe Mathieu-Daudé, 2019/12/17
- Re: [PATCH 00/14] chardev: Use QEMUChrEvent enum in IOEventHandler typedef, Philippe Mathieu-Daudé, 2019/12/17