Complete.Org: Mailing Lists: Archives: freeciv-dev: December 2002:
[Freeciv-Dev] Re: (PR#2566) PATCH: clean up handle_upgrade_unittype_requ
Home

[Freeciv-Dev] Re: (PR#2566) PATCH: clean up handle_upgrade_unittype_requ

[Top] [All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index] [Thread Index]
To: freeciv-dev <freeciv-dev@xxxxxxxxxxx>
Subject: [Freeciv-Dev] Re: (PR#2566) PATCH: clean up handle_upgrade_unittype_request
From: Jason Short <jdorje@xxxxxxxxxxxxxxxxxxxxx>
Date: 18 Dec 2002 16:11:42 -0500

On Wed, 2002-12-18 at 13:55, Guest via RT wrote:

> > can_upgrade_unittype is also a huge misnomer.  Do you want to 
> > provide a (separate) patch to rename it? 
>  
> I suppose I could, do you expect something like "successor_unittype"? 

Well, the name "can" implies it's a boolean return value - which is not
the case.  This confused me greatly when looking at your original patch.

I would suggest get_unittype_upgrade, but I'm no authority.  The return
type should also be changed to a unit type id.

jason



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