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

[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 17:59:15 +0100
Reply-to: rf13@xxxxxxxxxxxxxxxxxxxxxx

On Fri, Mar 08, 2002 at 03:43:41PM +0000, Ben Webb wrote:
> On Fri, 8 Mar 2002, Raimar Falke wrote:
> > I don't have access to a 64bit system here.
> What's wrong with the SourceForge compile farm?

Thanks for the hint. I get some of these:
  warning: cast increases required alignment of target type
If I understand this correct you shouldn't do
 char *p;
 int *q;
 void *r;

 q=(int *)p;

Since q may have stronger alignment requirements than p. Note that
is ok. So we have to change the type of a generic pointer from char *
to void *. Correct?!

> > So you may can provide patch or error messages.
> I did, but you only applied part of it. ;)
> See
>       Since then, lots of other gint/gpointer problems have crept in; 
> see attached patch.
> As for the size_t problem, this was partly addressed by 
> (If the "%zt" 
> format specifier isn't good enough for printing size_t's, e.g. from 
> sizeof, then I guess casting to unsigned long is a reasonable compromise.)

I'm for the unsigned long cast.


 email: rf13@xxxxxxxxxxxxxxxxx
 Make a software that is foolproof, and only fools will want to use it.

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