[Freeciv-Dev] Re: [Patch] city_option_set
[Top] [All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index] [Thread Index]
--- Jason Short <vze2zq63@xxxxxxxxxxx> wrote: > Raimar Falke wrote:
> > Encapsulated "pcity->city_options & (1<<x)" in a new function
> > city_option_set. Alternative names anyone?
>
> city_option_is_set. Or is_city_option_set.
>
> city_option_set sounds like an action, not a query...
I support Jason 100%
at first I thought that it is an action and only Jason's email forced
me to look at it more closely and discover that it's a test.
So city_option_set is a VERY misleading name.
G.
__________________________________________________
Do You Yahoo!?
Everything you'll ever need on one web page
from News and Sport to Email and Music Charts
http://uk.my.yahoo.com
|
|