[Freeciv-Dev] Re: Compaq Tru64 Unix Alpha platform - Building freeciv

[Raimar Falke <hawk@xxxxxxxxxxxxxxxxxxxxxxx>]

On Tue, May 14, 2002 at 12:38:57PM +0200, Raimar Falke wrote:
> On Sun, May 12, 2002 at 08:18:57PM +0200, Davide Pagnin wrote:
> As Ben already said: this is similar to GTK. There are macros which
> does the transformation. 

> Now we have to decide weither there are such macros in Xt

Looks like xaw doesn't have such functions. Or at least I haven't
found them.

> or we have to build our own.

Done.

> > 4) Another annoying warning present in many file was the "char
> > subscript",
> > related to the family of function like isalnum, isspace, toupper,
> > tolower,
> > isdigit. Alla of this are defined as: isspace(int c) and the c parameter
> > has to be convertible to an unsigned char or be the macro EOF.

The alpha host from cf.sf.net didn't report this error.

> > 5) There is a problem also with some asserts that eat a pointer as
> > the argument of they evaluation. I remember that some time ago Raimar
> > tried to convert a whole bunch of implicit pointer bool evaluation to
> > explicit, then all of this was undone, for some good reason.
> > Nontheless, I've spotted only 3 or 4 case in which alpha platform
> > require to change the actual code, so perhaps this is not a problem
> > for other reason. Feeding a pure pointer to an assert make the
> > compile warn of the convertion from pointer to integer of different
> > size (remember that int, the parameter of assert, in only 32 bit!).
> > 
> > Example:
> > assert(pplayer);
> > 
> > Solution, make the boolean evaluation explicit: assert(pplayer!=NULL);
> 
> Ack.

The alpha host from cf.sf.net didn't report this error.

        Raimar

-- 
 email: rf13@xxxxxxxxxxxxxxxxx
 "Make it idiot-proof and someone will make a better idiot."

64bit1.diff
Description: Text document