This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH v2] cpp/c: Add -Wexpansion-to-defined
- From: Sandra Loosemore <sandra at codesourcery dot com>
- To: Paolo Bonzini <bonzini at gnu dot org>, <gcc-patches at gcc dot gnu dot org>
- Cc: Manuel López-Ibáñez <lopezibanez at gmail dot com>
- Date: Wed, 10 Aug 2016 21:00:52 -0600
- Subject: Re: [PATCH v2] cpp/c: Add -Wexpansion-to-defined
- Authentication-results: sourceware.org; auth=none
- References: <1470823595-1471-1-git-send-email-bonzini@gnu.org>
On 08/10/2016 04:06 AM, Paolo Bonzini wrote:
===================================================================
--- gcc/doc/invoke.texi (revision 239276)
+++ gcc/doc/invoke.texi (working copy)
@@ -4914,6 +4914,12 @@
construct, known from C++, was introduced with ISO C99 and is by default
allowed in GCC@. It is not supported by ISO C90. @xref{Mixed Declarations}.
+@item -Wexpansion-to-defined
+@opindex Wexpansion-to-defined
+Warn whenever @samp{defined} is encountered in the expansion of a macro.
+(including the case where the macro is expanded by an @samp{#if} directive).
You've got too many periods in that sentence, and I'd get rid of the
parentheses, too:
Warn whenever @samp{defined} is encountered in the expansion of a macro,
including the case where the macro is expanded by an @samp{#if} directive.
-Sandra the nit-picky