Complete.Org: Mailing Lists: Archives: freeciv-dev: July 1999:
Re: [Freeciv-Dev] [PATCH] cosmetic fix at server/stdinhand.c
Home

Re: [Freeciv-Dev] [PATCH] cosmetic fix at server/stdinhand.c

[Top] [All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index] [Thread Index]
To: Markus Linnala <maage@xxxxxxxxx>
Cc: freeciv-dev@xxxxxxxxxxx
Subject: Re: [Freeciv-Dev] [PATCH] cosmetic fix at server/stdinhand.c
From: Greg Wooledge <wooledge@xxxxxxxxxxx>
Date: Mon, 5 Jul 1999 09:35:41 -0400

Markus Linnala (maage@xxxxxxxxx) wrote:

> Instead of using 0 use H_NONE. Also use & instead of + as I think it
> is best choice when you are messing with bitfields.

> -  int h[11] = { -1, 0, 0, H_RATES+H_TARGETS+H_HUTS,
> -             0, H_RATES+H_TARGETS+H_HUTS, 0, 0,
> -             0, 0, 0 };
> +  int h[11] = { -1, 
> +             H_NONE, 
> +             H_NONE, 
> +             H_RATES & H_TARGETS & H_HUTS,

You meant "|", not "&".

-- 
Greg Wooledge                    | Distributed.NET http://www.distributed.net/
wooledge@xxxxxxxxxxx             | because a CPU is a terrible thing to waste.
http://www.kellnet.com/wooledge/ |

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