On Fri, Jan 04, 2019 at 08:36:59AM +0000, Frediano Ziglio wrote: > Different tools (compilers or analysers) recognise missing break > in switch as possible problems. > Many recognise "fall through" comment to tell is wanted. a "fall through" comment (I think), "to tell it is wanted" Acked-by: Christophe Fergeau <cfergeau@xxxxxxxxxx> > > Signed-off-by: Frediano Ziglio <fziglio@xxxxxxxxxx> > --- > src/vdagentd/vdagentd.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/src/vdagentd/vdagentd.c b/src/vdagentd/vdagentd.c > index a2f0946..ec679fd 100644 > --- a/src/vdagentd/vdagentd.c > +++ b/src/vdagentd/vdagentd.c > @@ -965,6 +965,7 @@ static void daemonize(void) > case -1: > syslog(LOG_ERR, "fork: %m"); > retval = 1; > + // fall through > default: > udscs_destroy_server(server); > exit(retval); > -- > 2.20.1 > > _______________________________________________ > Spice-devel mailing list > Spice-devel@xxxxxxxxxxxxxxxxxxxxx > https://lists.freedesktop.org/mailman/listinfo/spice-devel
Attachment:
signature.asc
Description: PGP signature
_______________________________________________ Spice-devel mailing list Spice-devel@xxxxxxxxxxxxxxxxxxxxx https://lists.freedesktop.org/mailman/listinfo/spice-devel