From: Jim Blandy <jimb@red-bean.com>
To: gdb@sourceware.org
Subject: Fwd: -Wpointer-sign for GCC 4.1
Date: Wed, 18 Jan 2006 18:44:00 -0000 [thread overview]
Message-ID: <8f2776cb0601181040s4970ce9es15ebdcae50dccda2@mail.gmail.com> (raw)
In-Reply-To: <20060118173155.GM28863@synopsys.com>
The message below is kind of odd. We do use -Wall, so if the pointer
sign warning will be printed when -Wall is specified, we'll still need
to pass an explicit argument to disable it. Which doesn't exactly
take the decision out of our hands, as we were hoping.
I think we should decide, for ourselves, whether we think the warning
is helpful or not, and then not be demure about doing the necessary
GCC stuff to enable or disable it. Hoping GCC would answer the
question for us was dopey.
I think there's some documentation value in reserving gdb_byte for
binary blobs and char for host-format text. It wouldn't have been
worth it before, but at this point we've got fixes for almost all
those warnings in place; we can't get those hours back, so the
cost/benefit is different now. So I think we should continue to
request the warning.
---------- Forwarded message ----------
From: Joe Buck <Joe.Buck@synopsys.com>
Date: Jan 18, 2006 9:31 AM
Subject: Re: -Wpointer-sign for GCC 4.1
To: Mike Stump <mrs@apple.com>
Cc: Daniel Jacobowitz <drow@false.org>, gcc@gcc.gnu.org
On Wed, Jan 18, 2006 at 01:10:19AM -0800, Mike Stump wrote:
> On Jan 17, 2006, at 1:19 PM, Daniel Jacobowitz wrote:
> >Someone's informed Richard Stallman that this (annoying) warning
> >will not be
> >enabled by default in GCC 4.1.
>
> >But, it currently seems to be. Should it be turned off before the
> >release?
>
> The SC or Jim Wilson will know more than I.
>
> > If not, who told RMS it was? :-)
>
> Likewise.
The Emacs developers were unhappy, so RMS complained to the SC, and there
was a discussion. At first RMS just wanted the warning to appear only
with --pedantic or -Wpointer-sign, but he was convinced that it should
also appear with -Wall.
So the answer is that, after consulting with some of the affected people
(which is why you got mail, Mike) the SC told RMS that it would be
changed. If it hasn't been done yet, then it's a release blocker,
because it was a promise the SC made.
next parent reply other threads:[~2006-01-18 18:41 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <20060117211914.GA13055@nevyn.them.org>
[not found] ` <39BD9F7D-F512-40EA-804A-DBE9BAC97E2B@apple.com>
[not found] ` <20060118173155.GM28863@synopsys.com>
2006-01-18 18:44 ` Jim Blandy [this message]
2006-01-18 18:59 ` Daniel Jacobowitz
2006-01-18 19:01 ` Jim Blandy
2006-01-18 19:26 ` Daniel Jacobowitz
2006-01-18 20:29 ` Eli Zaretskii
2006-01-18 23:18 ` Mark Kettenis
2006-01-18 23:59 ` Daniel Jacobowitz
2006-01-19 1:01 ` Jim Blandy
2006-01-19 16:52 ` Eli Zaretskii
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=8f2776cb0601181040s4970ce9es15ebdcae50dccda2@mail.gmail.com \
--to=jimb@red-bean.com \
--cc=gdb@sourceware.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox