This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH] Fix cpp to match cpp.texi in respect to warnings
- To: Neil Booth <neil at daikokuya dot demon dot co dot uk>
- Subject: Re: [PATCH] Fix cpp to match cpp.texi in respect to warnings
- From: Jakub Jelinek <jakub at redhat dot com>
- Date: Tue, 10 Apr 2001 17:32:50 -0400
- Cc: gcc-patches at gcc dot gnu dot org, Zack Weinberg <zackw at Stanford dot EDU>
- References: <20010410170130.H1169@devserv.devel.redhat.com> <20010410221855.A12986@daikokuya.demon.co.uk>
- Reply-To: Jakub Jelinek <jakub at redhat dot com>
On Tue, Apr 10, 2001 at 10:18:55PM +0100, Neil Booth wrote:
> > cpp.texi sais about -Wcomment that it enables warning whenever
> > backslash-newline appears in a // comment, but in reality this warning is
> > issued unconditionally. Likewise, cpp.texi speaks about -Wwhite-space, but
> > the warning is unconditional too.
> > This patch changes cpp0 to match its documentation.
> > Bootstrapped on i386-redhat-linux, no regressions.
> > Ok to commit?
>
> The Wcomment line is fine for branch and mainline.
Ok, will commit it separately.
> As for -Wwhite-space, I never knew that existed. It must have been in cccp;
> I don't believe it was ever intended to be handled in cpplib. IIRC I just
> put in the cpplib warning because I noticed it was in one place and easy
> to do (after a discussion by Per about why it might be useful to allow
> such stuff without failing).
It seems it was brought in by a merge from Kenner's snapshot in September
1999, where -Wwhite-space was used for this case and also when there was
whitespace at end of line in a (now deprecated) multi-line string.
Jakub