On Wed, 2019-07-31 at 07:19 -0400, Neil Horman wrote: > On Tue, Jul 30, 2019 at 10:04:37PM -0700, Joe Perches wrote: > > fallthrough may become a pseudo reserved keyword so this only use of > > fallthrough is better renamed to allow it. > > > > Signed-off-by: Joe Perches <joe@xxxxxxxxxxx> > Are you referring to the __attribute__((fallthrough)) statement that gcc > supports? If so the compiler should by all rights be able to differentiate > between a null statement attribute and a explicit goto and label without the > need for renaming here. Or are you referring to something else? Hi. I sent after this a patch that adds # define fallthrough __attribute__((__fallthrough__)) https://lore.kernel.org/patchwork/patch/1108577/ So this rename is a prerequisite to adding this #define. > > diff --git a/net/sctp/sm_make_chunk.c b/net/sctp/sm_make_chunk.c [] > > @@ -2152,7 +2152,7 @@ static enum sctp_ierror sctp_verify_param(struct net *net, > > case SCTP_PARAM_SET_PRIMARY: > > if (net->sctp.addip_enable) > > break; > > - goto fallthrough; > > + goto unhandled; etc...
