[Freeciv-Dev] Re: version 1.11 revised schedule

[Thue <thue@xxxxxxx>]

Den tir, 25 jul 2000 skrev David Pfitzner:
> (Ahem, a rather late comment...)
> 
> On Wed, 28 Jun 2000 Jeff Mallatt wrote:
> > At 2000/06/27 18:50 , Kero van Gelder wrote:
> > >In that case: In the (AI)code the function ai_start_turn() is called twice.
> > >Is that intentional? The comments suggest it's not...
> > >(server/civserver.c version 1.167, line 491 & 521)
> > 
> > I believe it is intentional.
> 
> Agreed.  As another comment says, ai_start_turn() is a "misleading name 
> for manage_units", and the AI manages its units twice per turn (which 
> can be (and has been) argued to be a bug, but one not so easily fixed).
> 
> -- David

WIthout looking too much at the source or testing, is there any reason the
two calls couldn't be on the same side of the sniff_packets() loop,
removing the bug but (hopefully) keeping the functionality.

-Thue