On Sat, Apr 27, 2013 at 07:39:04PM +0100, Ramsay Jones wrote: > Sparse issues an "'junk_mode' not declared. Should it be static?" > warning. In order to suppress the warning, since this symbol does > not need more than file visibility, we simply add the static > modifier to it's declaration. > [...] > -enum { > +static enum { > JUNK_LEAVE_NONE, Thanks. This one was done by me, and yes it should be static. I think it is because I originally typed: enum junk_mode { ... }; static enum junk_mode junk_mode; and then later dropped the named type, but forgot to keep the static keyword. -Peff -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html