Message ID | 37c8748f80572a08f33e4ce354f93d61ddfd175b.1605896060.git.gustavoars@kernel.org |
---|---|
State | New |
Headers | show |
Series | Fix fall-through warnings for Clang | expand |
Hi all, It's been more than 3 months; who can take this, please? :) Thanks -- Gustavo On Fri, Nov 20, 2020 at 12:37:53PM -0600, Gustavo A. R. Silva wrote: > In preparation to enable -Wimplicit-fallthrough for Clang, fix a warning > by explicitly adding a break statement instead of letting the code fall > through to the next case. > > Link: https://github.com/KSPP/linux/issues/115 > Signed-off-by: Gustavo A. R. Silva <gustavoars@kernel.org> > --- > net/ax25/af_ax25.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/net/ax25/af_ax25.c b/net/ax25/af_ax25.c > index 269ee89d2c2b..2631efc6e359 100644 > --- a/net/ax25/af_ax25.c > +++ b/net/ax25/af_ax25.c > @@ -850,6 +850,7 @@ static int ax25_create(struct net *net, struct socket *sock, int protocol, > case AX25_P_ROSE: > if (ax25_protocol_is_registered(AX25_P_ROSE)) > return -ESOCKTNOSUPPORT; > + break; > #endif > default: > break; > -- > 2.27.0 >
diff --git a/net/ax25/af_ax25.c b/net/ax25/af_ax25.c index 269ee89d2c2b..2631efc6e359 100644 --- a/net/ax25/af_ax25.c +++ b/net/ax25/af_ax25.c @@ -850,6 +850,7 @@ static int ax25_create(struct net *net, struct socket *sock, int protocol, case AX25_P_ROSE: if (ax25_protocol_is_registered(AX25_P_ROSE)) return -ESOCKTNOSUPPORT; + break; #endif default: break;
In preparation to enable -Wimplicit-fallthrough for Clang, fix a warning by explicitly adding a break statement instead of letting the code fall through to the next case. Link: https://github.com/KSPP/linux/issues/115 Signed-off-by: Gustavo A. R. Silva <gustavoars@kernel.org> --- net/ax25/af_ax25.c | 1 + 1 file changed, 1 insertion(+)