re: CVS commit: src

[matthew green <mrg%eterna.com.au@localhost>]

"matthew green" writes:
> Module Name:	src
> Committed By:	mrg
> Date:		Sun Feb  3 03:19:32 UTC 2019
> 
> Modified Files:
> 	src/games/backgammon/teachgammon: teach.c
[ ... ]
> 	src/usr.sbin/wsmuxctl: wsmuxctl.c
> 
> Log Message:
> - add or adjust /* FALLTHROUGH */ where appropriate
> - add __unreachable() after functions that can return but won't in
>   this case, and thus can't be marked __dead easily

FYI:

there are a bunch of cases the fallthru checker still complains about
that should be valid constructs.  eg, these two both fail:

        case foo1: {  
                bar();
                /* FALLTHROUGH */
        }
        case foo2:

#ifdef SOMETHING
        case foo1:
                bar();
                /* FALLTHROUGH */
#endif
        case foo2:

AFAICT, the only construct accepted if where the comment is alone
(with or without spaces) on the immediately preceeding line.

i don't want to change code that has otherwise valid fallthru
comments, particularly the dozens of external files with both the
above constructs.

most of the changes to add or fix comments are already commited,
but more will be coming to -Wno-error this warning.


.mrg.