[PATCH] clean up warnings caused by slighly improper version of wordsize.h for ia64
Al Stone
ahs3 at fc.hp.com
Mon Mar 19 19:13:52 UTC 2007
On Sat, 2007-03-17 at 04:23 -0400, Mike Frysinger wrote:
> On Wednesday 14 March 2007, Al Stone wrote:
> > On Tue, 2007-03-13 at 23:14 -0400, Mike Frysinger wrote:
> > > On Tuesday 06 March 2007, Al Stone wrote:
> > > > Yet another compiler warning clean-up for ia64 ... in this case, the
> > > > warning was that __WORDSIZE_COMPAT32 was not defined. Added the proper
> > > > definition to libc/sysdeps/ia64/bits/wordsize.h to make the warning go
> > > > away.
> > >
> > > where is the warning ? pointless compat support just bloats uClibc and
> > > considering it doesnt even really work right now, dont want to do that
> > > for ia64 ;)
> >
> > Sigh. Another case where DODEBUG turns on EXTRA_WARNINGS and in turn
> > turns on a warning flag -- in this case, -Wundef. My mistake. It
> > should stay that way; with DODEBUG off, there is no warning.
>
> i think you misinterpreted ... i think the warning should be fixed, just not
> in the way you proposed ... and since i wasnt seeing any warnings on ia64, i
> asked how you managed to get one
> -mike
Indeed, I did misinterpret. So, using DODEBUG (and hence using
EXTRA_WARNINGS which adds -Wundef to CFLAGS) was how I got the
warning (just in case I wasn't clear :).
What would be a better way to fix this? Define a default
for __WORDSIZE_COMPAT in one of the common include files,
and let individual architectures change it as needed? Make
it a config variable?
--
Ciao,
al
----------------------------------------------------------------------
Al Stone Alter Ego:
Open Source and Linux R&D Debian Developer
Hewlett-Packard Company http://www.debian.org
E-mail: ahs3 at fc.hp.com ahs3 at debian.org
----------------------------------------------------------------------
More information about the uClibc
mailing list