[PATCH] Add more specific lexer flags for misc GNU extensions
Peter N Lewis
peter at stairways.com.au
Wed Aug 21 19:03:46 PDT 2013
On 22/08/2013, at 2:25 , Eli Friedman <eli.friedman at gmail.com> wrote:
> On Wed, Aug 21, 2013 at 6:10 AM, Peter N Lewis <peter at stairways.com.au> wrote:
> Here is batch two of the specific gnu flags. This patch adds the following flags:
>
> -Wgnu-zero-variadic-macro-arguments
> -Wgnu-token-pasting-comma
>
> These two are actually very closely related (the latter doesn't make sense without the former). We might want to consider putting them under the same flag.
I agree, and thought the same myself. Perhaps -Wgnu-variadic-macro to cover both? The "gnu" already implies it is a gnu extension, so -Wgnu-variadic-macro-extension would be redundant. But if we get to the point where we want to claim it as a "legitimate" extennsion, then -Wvariadic-macro would not work as an alternative, where -Wzero-variadic-macro-arguments and -Wtoken-pasting-comma do.
> -Wgnu-imaginary-constant
> -Wgnu-binary-literal
> -Wgnu-line-zero-directive
>
> This name is sort of confusing; it isn't really a "line-zero" directive.
Suggestions for an alternative? The error message (currently) is "#line directive with zero argument is a GNU extension" and happens when you have
#line 0
I wasn't sure -Wgnu-line-0-directive was a good idea. Any other ideas? Let me know and I'll redo the patch.
Peter.
--
Keyboard Maestro 6.2 now out - control Mail, reveal a file, format AppleScripts and more.
Keyboard Maestro <http://www.keyboardmaestro.com/> Macros for your Mac
<http://www.stairways.com/> <http://download.keyboardmaestro.com/>
More information about the cfe-commits
mailing list