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

[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]
To: per@xxxxxxxxxxx
Subject: [Freeciv-Dev] Re: (PR#6408) Remove unused parameter to wipe_unit_safe*
From: "Jason Short" <jdorje@xxxxxxxxxxxxxxxxxxxxx>
Date: Mon, 6 Oct 2003 08:55:06 -0700
Reply-to: rt@xxxxxxxxxxxxxx

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




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