Messages in this thread | | | Subject | Re: [PATCH] nfs: remove incorrect fallthrough label | From | Joe Perches <> | Date | Tue, 15 Sep 2020 16:29:41 -0700 |
| |
On Tue, 2020-09-15 at 15:57 -0700, Nick Desaulniers wrote: > There is no case after the default from which to fallthrough to. Clang > will error in this case (unhelpfully without context, see link below) > and GCC will with -Wswitch-unreachable. > > The previous commit should have just removed the comment. [] > diff --git a/fs/nfs/super.c b/fs/nfs/super.c [] > @@ -889,7 +889,6 @@ static struct nfs_server *nfs_try_mount_request(struct fs_context *fc) > default: > if (rpcauth_get_gssinfo(flavor, &info) != 0) > continue; > - fallthrough;
My preference would be to convert the fallthrough to a break here so if someone ever adds another label after default: for any reason, the code would still work as expected.
> } >
| |