Re: [ob] More warnings; Call for assistance

[Eli Zaretskii <eliz at gnu dot org>]

> Date: Tue, 17 Jan 2006 15:23:23 -0500
> From: Daniel Jacobowitz <drow@false.org>
> 
> On Tue, Jan 17, 2006 at 10:15:27PM +0200, Eli Zaretskii wrote:
> > > Date: Tue, 17 Jan 2006 10:17:31 -0500
> > > From: Daniel Jacobowitz <drow@false.org>
> > > 
> > > /space/fsf/commit/src/gdb/expprint.c:180: warning: pointer targets in passing argument 2 of 'current_language->la_printstr' differ in signedness
> > 
> > Is it really worth your trouble fixing this?  AFAIK, the next release
> > of GCC is supposed to turn this warning off by default, so why fix
> > something that ain't broken in the first place, if it's gonna bug only
> > those users who have GCC 4.0.x?
> 
> Do you have a reference for this GCC change?

I have this:

   http://lists.gnu.org/archive/html/emacs-devel/2005-12/msg01743.html

I don't know if this counts as a reference, but it's good enough for
me ;-)

> Also, do you think the warning is useless?

I think it's too pedantic; I think it's a false alarm 99.99% of the
time.  But if you think I'm wrong, please describe situations where
this warning would point to a real trouble.

> If you think it's useless, I'll just stop, and we'll go with Mark's
> patch to disable it.

Hmm.. were you fixing only this precise warning?  I thought I've seen
other, more serious warnings as well fixed by your recent changes.