Complete.Org: Mailing Lists: Archives: freeciv-dev: May 2004:
[Freeciv-Dev] (PR#8672) punit->occupy has no value at the server
Home

[Freeciv-Dev] (PR#8672) punit->occupy has no value at the server

[Top] [All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index] [Thread Index]
To: undisclosed-recipients: ;
Subject: [Freeciv-Dev] (PR#8672) punit->occupy has no value at the server
From: "Jason Short" <jdorje@xxxxxxxxxxxxxxxxxxxxx>
Date: Tue, 4 May 2004 10:48:11 -0700
Reply-to: rt@xxxxxxxxxxx

<URL: http://rt.freeciv.org/Ticket/Display.html?id=8672 >

Originally punit->client.occupy was created so the client could track if 
units were occupied.  It was a boolean.

It was changed to punit->occupy and turned into an integer.  I don't 
know why.  The server still doesn't use this value.

However the AI code has started using the value.  This won't work since 
it's always 0 at the server.

The quick fix is that the AI should call get_transporter_occupancy() 
instead.  However this function is slow.  Better would be to update the 
punit->occupy value at the server.  It should be renamed 
(punit->occupancy?) and updated in 
load_unit_onto_transporter/unload_unit_from_transporter.

jason




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