[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