[Freeciv-Dev] [patch #1617] Release script objects for removed units and cities

Update of patch #1617 (project freeciv): Status: Ready For Test => Done Open/Closed:Open => Closed ___ Reply to this item at:

[Freeciv-Dev] [patch #1617] Release script objects for removed units and cities

Follow-up Comment #6, patch #1617 (project freeciv): Yet another version. To be consistent across save/load we have to restore even 'Nonexistent' values. Also removing the script_is_exported_object_valid function. It is mostly for debugging. Patches for trunk and 2.2 (file #8956, file #8957)

[Freeciv-Dev] [patch #1617] Release script objects for removed units and cities

Update of patch #1617 (project freeciv): Planned Release: 2.3.0 => 2.2.1, 2.3.0 ___ Follow-up Comment #5: Patch for 2.2 branch attached, adding same object removal and new API for :exists() methods.

[Freeciv-Dev] [patch #1617] Release script objects for removed units and cities

Follow-up Comment #4, patch #1617 (project freeciv): Updating the patch. This makes the mechanism much better. New summary: We change the type of the userdata to a marker class called 'Nonexistent' [when a freeciv object such as city or unit is free'd]. Nonexistent has one single method :exist

[Freeciv-Dev] [patch #1617] Release script objects for removed units and cities

Follow-up Comment #3, patch #1617 (project freeciv): At the moment an array of 32 players is allocated at the start. So the number of players is fixed. Perhaps someday the players will be a dynamically allocated list ... But you are right. Especially in that case, a player will never be deleted

[Freeciv-Dev] [patch #1617] Release script objects for removed units and cities

Follow-up Comment #2, patch #1617 (project freeciv): Ah ok. I'll look at player in a followup patch. It seems players are never released, but they are put into an unusable state after being removed so it's similar. Attached patch has minor update for change in tolua_outside patch. (file #8929)

[Freeciv-Dev] [patch #1617] Release script objects for removed units and cities

Follow-up Comment #1, patch #1617 (project freeciv): I think something similar is needed for struct player as it is also exported to lua and can be freed ___ Reply to this item at: _

[Freeciv-Dev] [patch #1617] Release script objects for removed units and cities

Update of patch #1617 (project freeciv): Depends on: => bugs #13709 ___ Reply to this item at: ___ Meddelande

[Freeciv-Dev] [patch #1617] Release script objects for removed units and cities

URL: Summary: Release script objects for removed units and cities Project: Freeciv Submitted by: englabenny Submitted on: söndag 2010-04-11 den 22:10 Category: general Priority