[Freeciv-Dev] Re: (PR#6408) Remove unused parameter to wipe_unit_safe*
[Top] [All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index] [Thread Index]
Per I. Mathisen wrote:
> On Mon, 6 Oct 2003, Jason Short wrote:
>
>>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?
>
>
> We can at least remove one of them. Done in this expanded version of the
> patch. Three unnecessary functions removed.
> - wipe_unit_spec_safe(punit, NULL, FALSE);
> + wipe_unit_spec_safe(punit, FALSE);
> } unit_list_iterate_end;
Wipe_unit_spec_safe should just be wipe_unit_spec? What does "spec" mean?
jason
|
|