Re: [PATCH 2/5] Fix some "enum value 'SYM_...' not handled in switch" warnings

[Ramsay Jones <ramsay@ramsay1.demon.co.uk>]

Christopher Li wrote:
> OK. I don't want to list all the enumerate value here just for the
> sake of gcc warnings.
> It makes the code ugly. Nor do I want to change the gcc flags used to compile
> sparse. If the newest gcc still complain on those. I would rather add
> the blank default
> to make it clean. Since latest gcc doesn't issue warning on those. I
> think it is fine to
> leave it as it is.
> 

OK, no problem. I can maintain this in my cygwin repo, along with the
__sentinel__ attribute patch. (As before, I don't need this on Linux).

Just FYI, I had planned to fix this by implementing a local makefile
config file (like config.mak in git), so that I could fix this issue
locally ("out of tree").

However, I was (pleasantly) surprised to find that I'd been beaten to
the punch by Samuel Bronson in commit 8d86d0e.

So I tried this local.mk file:

    OS=cygwin
    CFLAGS+=-Wno-switch-enum

This lead to the invocation of gcc I was hoping for; something like:

    gcc ... -Wall ... -Wno-switch-enum parse.c

which I had expected to suppress the warnings. It didn't. *Ho-Hum* ;-)

ATB,
Ramsay Jones