Re: [PATCH] net: rxrpc: mark expected switch fall-throughs

From: David Howells
Date: Thu Oct 19 2017 - 17:17:07 EST


Gustavo A. R. Silva <garsilva@xxxxxxxxxxxxxx> wrote:

> + /* fall through */

All new comments in rxrpc should begin with a capital letter; I'm switching to
this as I modify the lines with comments on. Fix checkpatch or gcc or
whatever takes -Wimplicit-fallthrough to stop being silly.

> - /* Fall through */
> -
> + /* fall through */

No. Firstly, it should be 'F'; secondly, don't remove the blank line - it's
there for a reason.

> + /* fall through */

Capital 'F'.

> -
> + /* fall through */

Don't remove the blank line. Capital 'F'.

> + /* fall through */

Capital 'F'.

David