Complete.Org: Mailing Lists: Archives: freeciv-dev: September 2001:
[Freeciv-Dev] Re: [Patch] MAPSTEP
Home

[Freeciv-Dev] Re: [Patch] MAPSTEP

[Top] [All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index] [Thread Index]
To: "Ross W. Wetmore" <rwetmore@xxxxxxxxxxxx>
Cc: Gregory Berkolaiko <gberkolaiko@xxxxxxxxxxx>, freeciv development list <freeciv-dev@xxxxxxxxxxx>
Subject: [Freeciv-Dev] Re: [Patch] MAPSTEP
From: Raimar Falke <hawk@xxxxxxxxxxxxxxxxxxxxxxx>
Date: Sun, 23 Sep 2001 09:00:29 +0200
Reply-to: rf13@xxxxxxxxxxxxxxxxxxxxxx

On Sun, Sep 23, 2001 at 12:10:41AM -0400, Ross W. Wetmore wrote:
> >From the diff, it looks like you just rediscovered the missing memset
> in find_a_direction. 

Yes.

>   /* stack garbage is often >best

You can't trust stack garbage. It is simply uninitialized.

> - clear it now, not in every special case */
>   memset(d,0,sizeof(d));
>    
> 
> Or did I miss something else. There were only a couple places in the 
> corecleanup where your patch took, but I carefully haven't converted the GUI 
> where there seemed to be more cases.
> 
> FYI: the corecleanup is stable wrt autogames, I updated the various
>      iteration functions in map.h to use it (using rotational order).

What are you talking about I didn't find anything in the incoming.

        Raimar

-- 
 email: rf13@xxxxxxxxxxxxxxxxx
  The trick is to keep breathing.


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