Complete.Org: Mailing Lists: Archives: freeciv-dev: October 2003:
[Freeciv-Dev] (PR#6408) Remove unused parameter to wipe_unit_safe* 		Home

[Freeciv-Dev] (PR#6408) Remove unused parameter to wipe_unit_safe*

[Top] [All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index] [Thread Index]
To: per@xxxxxxxxxxx
Subject: [Freeciv-Dev] (PR#6408) Remove unused parameter to wipe_unit_safe*
From: "Jason Short" <jdorje@xxxxxxxxxxxxxxxxxxxxx>
Date: Mon, 6 Oct 2003 06:54:22 -0700
Reply-to: rt@xxxxxxxxxxxxxx 
[per - Mon Oct  6 13:49:40 2003]:

> Backport of an older patch. This removes the now unused (and always ugly)
> iterator parameter to wipe_unit_safe() and wipe_unit_spec_safe(). It has
> been unnecessary since Jason rewrote the latter function.

Nice.

There are a lot of functions in the server for removing units:
wipe_unit, wipe_unit_safe, wipe_unit_spec_safe, server_remove_unit(). 
Can we rewrite these to just be one (safe) function, possibly with a
(local) helper function?

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