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 -- To unsubscribe from this list: send the line "unsubscribe linux-sparse" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html