Yeah, it worked for those too.  I saw you made the change already, so
nothing for me to do.

By the way, my wife is almost 39 weeks pregnant, so I may just disappear
for a couple weeks when the baby comes.


On Fri, May 16, 2014 at 9:20 PM, Joss Whittle <notifications@github.com>wrote:

> In flint/Checks.cpp:
>
> > +				pos += 2;
> > +				continue;
> > +			}
> > +
> > +			if (atSequence(tokens, pos, unnamedNamespace)) {
> > +				pos += 2;
> > +			}
> > +
> > +			const Token &token = tokens[pos];
> > +			if (isTok(token, TK_LCURL)) {
> > +				pos = skipBlock(tokens, pos);
> > +				continue;
> > +			}
> > +
> > +			if (isTok(tokens[pos], TK_STATIC)) {
> > +				lintWarning(errors, tokens[pos], "Don't use static variables at global or namespace scopes.");
>
> Does this fire for static functions as well? Suggest changing the message
> to something like
>
> Don't use static at global and namespace scopes.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/L2Program/FlintPlusPlus/pull/34/files#r12765152>
> .
>