Complete.Org: Mailing Lists: Archives: freeciv-dev: March 2002:
[Freeciv-Dev] Re: Economy report (PR#1307)
Home

[Freeciv-Dev] Re: Economy report (PR#1307)

[Top] [All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index] [Thread Index]
To: Ben Webb <ben@xxxxxxxxxxxxxxxxxxxxxx>
Cc: jdorje@xxxxxxxxxxxxxxxxxxxxx, freeciv-dev@xxxxxxxxxxx
Subject: [Freeciv-Dev] Re: Economy report (PR#1307)
From: Raimar Falke <hawk@xxxxxxxxxxxxxxxxxxxxxxx>
Date: Fri, 8 Mar 2002 16:11:13 +0100
Reply-to: rf13@xxxxxxxxxxxxxxxxxxxxxx

On Fri, Mar 08, 2002 at 02:26:22PM +0000, Ben Webb wrote:
> On Fri, 8 Mar 2002, Jason Short wrote:
> 
> > Ben Webb wrote:
> > >   That'd be nice, except that then Freeciv would fail to build on 
> > > an Alpha...
> > 
> > Raimar means it for local compilation, i.e.
> > 
> >    make CFLAGS="-Wshadow -Werror"
> 
>       "Local" compilation? My local machine is an Alpha, so this still 
> wouldn't work. ;)
> 
> > have imagined there were a lot of other shadowed variables causing 
> > things not to compile at all like this, but maybe Raimar's fixed these now.
> 
>       The main problem on Alpha is not shadowed variables, but current 
> builds churn out buckets of warnings from code that assumes sizeof(int) == 
> sizeof(void *). (gui-gtk is the main offender here.) There is also the 
> assumption that sizeof(int) == sizeof(size_t), mainly in the agents code, 
> IIRC. Adding -Werror would change these to errors, and cause the build to 
> fail.

I don't have access to a 64bit system here. So you may can provide
patch or error messages.

        Raimar

-- 
 email: rf13@xxxxxxxxxxxxxxxxx
 "Sit, disk, sit. Good boy. Now spin up. Very good. Here's a netscape
  cookie for you. Fetch me some data. Come on, you can do it. No, not that
  data. Bad disk. Bad." 
    -- Calle Dybedahl, alt.sysadmin.recovery


[Prev in Thread] Current Thread [Next in Thread]