[Freeciv-Dev] Re: (PR#4676) clean up pcity->occupied usage

["Gregory Berkolaiko" <Gregory.Berkolaiko@xxxxxxxxxxxx>]

On Thu, 24 Jul 2003, Jason Short wrote:

> This patch cleans this usage up in two ways:
> 
> 1.  Instead of pcity->occupied it's now pcity->client.occupied.  This 
> makes it harder to accidentally use it at the server (which was one of 
> the original bugs with this code), and is a prelude for more 
> client-side-only fields (see PR#3771).  Of course this means all users 
> have to be changed.

Good idea!

> 2.  Lots of comments added about the client-side usage of this field. 
> I've satisfied myself that the client side of the code is correct, and 
> put this explanation into the comments.

Well done!

G.