[freeciv-ai] Re: patch/rfc: wrap activity
[Top] [All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index] [Thread Index]
On Sat, 23 Nov 2002, Ross W. Wetmore wrote:
> If the checks make sense, then push them down to the appropriate level
> and do them there.
> Do not clutter the code with Tom, Dick and Harry functions that all
> patch up the underlying core routine in subtly different ways at the
> expense of layered function calls and code duplication.
This might make sense, I agree. The original reason for making this
wrapper was to packetify the activity changes, but if we decide not to
move this AI code to the client anyway, as Greg has suggested, then we
don't need it.
So maybe we should take the move-to-client discussion first.