Complete.Org: Mailing Lists: Archives: freeciv-dev: September 2001:
[Freeciv-Dev] Re: [Patch] Remove goto_zoc_ok and dir_ok
Home

[Freeciv-Dev] Re: [Patch] Remove goto_zoc_ok and dir_ok

[Top] [All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index] [Thread Index]
To: Thue <thue@xxxxxxx>
Cc: freeciv development list <freeciv-dev@xxxxxxxxxxx>
Subject: [Freeciv-Dev] Re: [Patch] Remove goto_zoc_ok and dir_ok
From: Raimar Falke <hawk@xxxxxxxxxxxxxxxxxxxxxxx>
Date: Tue, 18 Sep 2001 20:04:19 +0200
Reply-to: rf13@xxxxxxxxxxxxxxxxxxxxxx

On Tue, Sep 18, 2001 at 07:32:39PM +0200, Thue wrote:
> On Tuesday 18 September 2001 19:15, Raimar Falke wrote:
> > I claim that the attached patch will not change anything on the
> > behavior of the code. It turns out that goto_zoc_ok has never
> > worked. Studing goto_zoc_ok reveals that it returns 1 if the
> > can_step_taken_wrt_to_zoc is true and 0 otherwise.
> >
> > Thue: you made these changes. Maybe you know something I missed or
> > can fix it.
> >
> >     Raimar
> 
> Evil bug - the last return in goto_zoc_ok should be a "return 1;".

So the question is: how is it possible that this wasn't found yet? To
show the impact of the "return 1" and for futher fixing a testcase is
really needed.

        Raimar

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


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