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

2010-04-12 Thread Matthias Pfafferodt
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: http://gna.org/patch/?1617

[Freeciv-Dev] [patch #1582] UTF-8 utilities

2010-04-12 Thread pepeto
Update of patch #1582 (project freeciv): Status: Ready For Test = None ___ Follow-up Comment #5: ATM there is an issue with MAX_LEN_NAME. This is supposed to limit city names, for instance,

[Freeciv-Dev] [bug #15810] Sanity check should use fc_assert()

2010-04-12 Thread pepeto
Update of bug #15810 (project freeciv): Status: Ready For Test = Fixed Open/Closed:Open = Closed ___ Reply to this item at:

[Freeciv-Dev] [bug #15816] add a comment about a needed empty line in the nation.ruleset file

2010-04-12 Thread pepeto
Update of bug #15816 (project freeciv): Assigned to:None = pepeto ___ Follow-up Comment #5: I intent to commit my fixes in the listed branch by 'Planned Release'.

[Freeciv-Dev] [bug #15841] API: Do not use tolua_outside

2010-04-12 Thread Ulrik Sverdrup
Follow-up Comment #1, bug #15841 (project freeciv): Updated patch, this way it's even better/easier. Will commit soon. (file #8928) ___ Additional Item Attachment: File name: 2010041201-API-Do-not-use-tolua_outside.patch Size:6 KB

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

2010-04-12 Thread Ulrik Sverdrup
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

2010-04-12 Thread Matthias Pfafferodt
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] [bug #15857] Fix buggy find_transport_from_tile

2010-04-12 Thread Ulrik Sverdrup
URL: http://gna.org/bugs/?15857 Summary: Fix buggy find_transport_from_tile Project: Freeciv Submitted by: englabenny Submitted on: måndag 2010-04-12 den 14:05 Category: general Severity: 3 - Normal

[Freeciv-Dev] [bug #15841] API: Do not use tolua_outside

2010-04-12 Thread Ulrik Sverdrup
Update of bug #15841 (project freeciv): Status: Ready For Test = Fixed Open/Closed:Open = Closed ___ Reply to this item at:

[Freeciv-Dev] [bug #13709] Switch from toluaxx to tolua-5.1b

2010-04-12 Thread Ulrik Sverdrup
Update of bug #13709 (project freeciv): Status: Ready For Test = Fixed Open/Closed:Open = Closed ___ Follow-up Comment #10: Switched to

[Freeciv-Dev] [bug #13709] Switch from toluaxx to tolua-5.1b

2010-04-12 Thread Matthias Pfafferodt
Follow-up Comment #11, bug #13709 (project freeciv): I get errors trying to compile S2_2 and trunk S2_2: dependencies/Makefile.am:3: required directory dependencies/tolua-5.1 does not exist utility/Makefile.am:60: `%'-style pattern rules are a GNU make extension configure.ac:942: required file

[Freeciv-Dev] [patch #1574] add a player to a running game

2010-04-12 Thread Matthias Pfafferodt
Follow-up Comment #10, patch #1574 (project freeciv): I think the interface could be create_player(player name, nation name) with both arguments as string. Is it possible to set nation name to nil to get a NULL pointer in freeciv? This way it would be possible to define a nation but also a

[Freeciv-Dev] [patch #1582] UTF-8 utilities

2010-04-12 Thread Matthias Pfafferodt
Follow-up Comment #6, patch #1582 (project freeciv): A problem would be for the packets. The packet size seems to need a byte limit, not in characters. This could be solved by multiplying all packet string sizes by 4(?) to take into account multicharacter bytes

[Freeciv-Dev] [bug #13709] Switch from toluaxx to tolua-5.1b

2010-04-12 Thread Ulrik Sverdrup
Update of bug #13709 (project freeciv): Status: Fixed = In Progress Planned Release: 2.3.0 = 2.2.1, 2.3.0 ___ Follow-up Comment #14: The trunk error

[Freeciv-Dev] [bug #15860] Fix compiler warning in tolua-5.1

2010-04-12 Thread Ulrik Sverdrup
URL: http://gna.org/bugs/?15860 Summary: Fix compiler warning in tolua-5.1 Project: Freeciv Submitted by: englabenny Submitted on: måndag 2010-04-12 den 20:48 Category: general Severity: 3 - Normal

[Freeciv-Dev] [bug #13709] Switch from toluaxx to tolua-5.1b

2010-04-12 Thread Ulrik Sverdrup
Update of bug #13709 (project freeciv): Status: In Progress = Fixed ___ Follow-up Comment #17: The compilation error is tracked in a separate bug #15860

[Freeciv-Dev] [bug #15860] Fix compiler warning in tolua-5.1

2010-04-12 Thread pepeto
Follow-up Comment #2, bug #15860 (project freeciv): This is a compilation failure, commit directly? If this is the case (not for me at least), you could commit it immediately. ___ Reply to this item at: http://gna.org/bugs/?15860

[Freeciv-Dev] [patch #1611] Scripting: Hide Lua module 'debug'

2010-04-12 Thread Ulrik Sverdrup
Update of patch #1611 (project freeciv): Planned Release: 2.3.0 = 2.2.1, 2.3.0 ___ Follow-up Comment #1: This should go into both 2.2 and trunk, since the change is easily available to make in 2.2 as

[Freeciv-Dev] [patch #1605] API: Add Tile:city() and Tile:num_units() accessors

2010-04-12 Thread Ulrik Sverdrup
Update of patch #1605 (project freeciv): Status: Ready For Test = Done Open/Closed:Open = Closed ___ Reply to this item at:

[Freeciv-Dev] [patch #1606] API: Add Tile, Player:units_iterate(), and Player:cities_iterate()

2010-04-12 Thread Ulrik Sverdrup
Update of patch #1606 (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

2010-04-12 Thread Ulrik Sverdrup
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

[Freeciv-Dev] [patch #1622] Karelian nation

2010-04-12 Thread J.M. Maalderink
URL: http://gna.org/patch/?1622 Summary: Karelian nation Project: Freeciv Submitted by: mixcoatl Submitted on: Monday 04/12/2010 at 22:38 Category: rulesets Priority: 5 - Normal Status:

[Freeciv-Dev] [bug #15858] Game crashes on startup

2010-04-12 Thread David Lowe
Follow-up Comment #1, bug #15858 (project freeciv): I suspect you're using the binary provided by Napoleon XIV? The rest of us have moved on to 2.2.0 by now, but there's no binary yet. If you install the 'fink' build system you can test out my 2.2.0 package. Anyway, that binary has a known