[freeciv-ai] Re: fix for strange goto code
[Top] [All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index] [Thread Index]
Actually there was a bug in goto_is_sane, annoying little bug.
Fix is attached. Check if it works without your correction to
do_unit_goto.
Your correction still makes sense though...
On Thu, 7 Nov 2002, Per I. Mathisen wrote:
> On Thu, 7 Nov 2002, Gregory Berkolaiko wrote:
> > You change do_unit_goto.The changes are sane, but what is the caller of
> > this function that was having problems.Please give more info.
>
> To reproduce: Watch the AI ship a settler overseas, then watch it abandon
> that ship permanently at the point where it drops off the settler.
>
> I tracked the error down to this place, and the patch fixes it. However,
> _why_ it fixes it is a good question. This is another case where I just
> have to ask - why did this code ever work? There must have been a bug in
> goto_is_sane() that you fixed that eliminated the bug that I just
> discovered...
>
> - Per
>
>
goto_is_sane_corr.diff
Description: Text document
|
|