[Freeciv-Dev] Re: [Patch] moves_left
[Top] [All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index] [Thread Index]
--- Raimar Falke <hawk@xxxxxxxxxxxxxxxxxxxxxxx> wrote: >
> I assume you guess it: moves_left is uses as a boolean comparison. Bla
> bla bla.
>
> Raimar
>
> - if (!punit->moves_left) return 1;
> + if (punit->moves_left == 0) return 1;
to be even more fail safe, you can do
if (punit->moves_left <= 0 ) return 1;
BTW, this all is very nice, but you are fixing the code that is already
quite ok. How about doing something more challenging / useful ?
Best,
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
|
|