Complete.Org: Mailing Lists: Archives: freeciv-dev: March 2003:
[Freeciv-Dev] Re: (PR#2370) Path finding
Home

[Freeciv-Dev] Re: (PR#2370) Path finding

[Top] [All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index] [Thread Index]
To: rf13@xxxxxxxxxxxxxxxxx
Subject: [Freeciv-Dev] Re: (PR#2370) Path finding
From: "Gregory Berkolaiko" <Gregory.Berkolaiko@xxxxxxxxxxxx>
Date: Tue, 4 Mar 2003 03:50:35 -0800
Reply-to: rt@xxxxxxxxxxxxxx

On Mon, 3 Mar 2003, Raimar Falke wrote:

> On Mon, Mar 03, 2003 at 05:00:29AM -0800, Gregory Berkolaiko wrote:
> > > > +  enum direction8 dir_to_next_pos;     /* Unsed only in struct_path */
> > > 
> > > It should be set to an invalid value otherwise.
> > 
> > Is
> >     #ifdef DEBUG 
> > ok with you?
> 
> This is ugly. And the write comes for free since this region is
> already cached.

What do you mean already cached?

And now a quote:
=============
Jason> Moreover, spurious initialization may conceal other errors (like
Jason> accessing invalid data) that could otherwise be more easily caught
Jason> (by valgrind, for instance).

Raimar> Yes.
=============

G.





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