Re: [Freeciv-Dev] Please test trunk!

2010-07-11 Thread Matthias Pfafferodt
Am Sunday 11 July 2010 05:48:23 schrieb Karl Goetz: On Sat, 10 Jul 2010 13:46:29 +0200 Matthias Pfafferodt matthias.pfaffer...@mapfa.de wrote: Am Saturday 10 July 2010 13:10:28 schrieb Matthias Pfafferodt: Hello all developers of freeciv, today I submitted some big changes to trunk:

[Freeciv-Dev] [patch #1742] [players] cleanup game_(init|free|reset)

2010-07-11 Thread Matthias Pfafferodt
: identical functions for server / client ___ File Attachments: --- Date: Sonntag 11.07.2010 um 10:18 Name: 20100711-05-trunk-cleanup-game_-init-free-reset.patch Size: 2kB By: syntron http

[Freeciv-Dev] [patch #1741] [players] use player_slot_count

2010-07-11 Thread Matthias Pfafferodt
Update of patch #1741 (project freeciv): Summary: use player_slot_count = [players] use player_slot_count ___ Reply to this item at: http://gna.org/patch/?1741 ___

[Freeciv-Dev] [patch #1743] [players] dynamically allocate player-diplstates

2010-07-11 Thread Matthias Pfafferodt
: ___ File Attachments: --- Date: Sonntag 11.07.2010 um 10:19 Name: 20100711-06-trunk-dynamically-allocate-player-diplstates.patch Size: 76kB By: syntron http://gna.org/patch

[Freeciv-Dev] [patch #1744] [players] rename struct player-ai_data to ai_common

2010-07-11 Thread Matthias Pfafferodt
: * can't be confused with aidata and struct ai_data * move initialization of ai_common.love into player.c ___ File Attachments: --- Date: Sonntag 11.07.2010 um 10:19 Name: 20100711-07

[Freeciv-Dev] [patch #1745] [players] dynamically allocate ai_data

2010-07-11 Thread Matthias Pfafferodt
: ___ File Attachments: --- Date: Sonntag 11.07.2010 um 10:20 Name: 20100711-08-trunk-dynamically-allocate-ai_data.patch Size: 47kB By: syntron http://gna.org/patch/download.php?file_id=9484

[Freeciv-Dev] [patch #1732] [Metaticket] increase number of players

2010-07-11 Thread Matthias Pfafferodt
Update of patch #1732 (project freeciv): Depends on: = patch #1739 ___ Reply to this item at: http://gna.org/patch/?1732 ___ Nachricht

[Freeciv-Dev] [patch #1732] [Metaticket] increase number of players

2010-07-11 Thread Matthias Pfafferodt
Follow-up Comment #1, patch #1732 (project freeciv): player pointers patches; there is an error wrt the editor: it crashes if one tries to change the player. I expect that there are some assumptions about the player struct (only patch #1739 needed) 1/6: [players] use pointers for players (patch

[Freeciv-Dev] [patch #1698] Dahomey

2010-07-11 Thread Ann
Follow-up Comment #2, patch #1698 (project freeciv): It looks a lot better than my version. I think the central figure and the swords are nice. The heads look a bit more headlike than in the original, but I can't imagine anyone will see that when it has been shrunk.

[Freeciv-Dev] [patch #1744] [players] rename struct player-ai_data to ai_common

2010-07-11 Thread Matthias Pfafferodt
: 20100711-07v2-trunk-rename-struct-player-ai_data-to-ai_common.patch Size:108 KB ___ Reply to this item at: http://gna.org/patch/?1744 ___ Nachricht geschickt von/durch Gna! http://gna.org

[Freeciv-Dev] [patch #1746] [players] send diplstate as separate network packages

2010-07-11 Thread Matthias Pfafferodt
URL: http://gna.org/patch/?1746 Summary: [players] send diplstate as separate network packages Project: Freeciv Submitted by: syntron Submitted on: Sonntag 11.07.2010 um 12:44 Category: general Priority:

[Freeciv-Dev] [patch #1732] [Metaticket] increase number of players

2010-07-11 Thread Matthias Pfafferodt
Update of patch #1732 (project freeciv): Depends on: = patch #1746 ___ Reply to this item at: http://gna.org/patch/?1732 ___ Nachricht

Re: [Freeciv-Dev] Please test trunk!

2010-07-11 Thread Marko Lindqvist
On 11 July 2010 11:13, Matthias Pfafferodt matthias.pfaffer...@mapfa.de wrote: Am Sunday 11 July 2010 05:48:23 schrieb Karl Goetz: On Sat, 10 Jul 2010 13:46:29 +0200 Matthias Pfafferodt matthias.pfaffer...@mapfa.de wrote: * new savegame format (use 'set saveversion -1' to get a savegame

Re: [Freeciv-Dev] Please test trunk!

2010-07-11 Thread Matthias Pfafferodt
old games will refuse to load the new save file due to a required capability (I hope '+version2' does this?!). If you want to use such a game with an old version you have to load it in 2.2.99+ and save it in the old format. Just to clarify: This change was committed to trunk only, so

[Freeciv-Dev] [bug #16245] Every government is allowed at start

2010-07-11 Thread Lavrentij P Berija
Follow-up Comment #4, bug #16245 (project freeciv): The patch definitely works. ___ Reply to this item at: http://gna.org/bugs/?16245 ___ Message sent via/by Gna! http://gna.org/

[Freeciv-Dev] [patch #1744] [players] rename struct player-ai_data to ai_common

2010-07-11 Thread Matthias Pfafferodt
Follow-up Comment #2, patch #1744 (project freeciv): player-real_embassy could also be moved into diplstate. Thus only one part of the player package is per (defined) player ___ Reply to this item at: http://gna.org/patch/?1744

[Freeciv-Dev] [patch #1746] [players] send diplstate as separate network packages

2010-07-11 Thread Matthias Pfafferodt
Follow-up Comment #1, patch #1746 (project freeciv): player-real_embassy could also be moved into diplstate. Thus only one part of the player package is per (defined) player (see also patch #1744) ___ Reply to this item at:

[Freeciv-Dev] [patch #1744] [players] rename struct player-ai_data to ai_common

2010-07-11 Thread Marko Lindqvist
Follow-up Comment #3, patch #1744 (project freeciv): = love into player:player_diplstate This affects only AI players, right? It should remain in structure clearly marked as AI stuff. I'm against polluting other structures with parts of AI stuff, especially with something that is only

[Freeciv-Dev] [patch #1746] [players] send diplstate as separate network packages

2010-07-11 Thread Marko Lindqvist
Follow-up Comment #2, patch #1746 (project freeciv): Each diplstate should definitely sent as separate package and not all of them as part of one big player package. This will also remove one more limit from maximum number of players; player packet will not grow beyond maximum packet size.

[Freeciv-Dev] [patch #1700] Freeciv 2.2.1 patches for Mac OS X 10.6.3

2010-07-11 Thread Marko Lindqvist
Follow-up Comment #1, patch #1700 (project freeciv): Please make separate ticket for each separate issue in the future. 1, 2, 7 and 8 ftwl client is not maintained and is likely to be completely removed soon, so it's likely nobody bothers to handle these patches. 3-6 Sounds like real problem.

[Freeciv-Dev] [bug #16249] Can't build trunk with -Werror: format not a string literal and no format arguments

2010-07-11 Thread Marko Lindqvist
Follow-up Comment #3, bug #16249 (project freeciv): (Perhaps a newish-GCC thing?) Hardened GCC thing. I fixed dozens of such things when I first tried to build freeciv with hardened gcc. ___ Reply to this item at:

[Freeciv-Dev] [bug #16249] Can't build trunk with -Werror: format not a string literal and no format arguments

2010-07-11 Thread Matthias Pfafferodt
Follow-up Comment #4, bug #16249 (project freeciv): yes, I now remember the discussion; but as I don't see this, I easily forget it ... ___ Reply to this item at: http://gna.org/bugs/?16249

[Freeciv-Dev] [patch #1739] [players] use pointers for players

2010-07-11 Thread Matthias Pfafferodt
Follow-up Comment #6, patch #1739 (project freeciv): a small analysis of the memory usage of freeciv (using pmap) naming convention: * freeciv server: log_server-version-# player-1 = connected + started * freeciv client (gtk): log_client-version-# player-1 = connected + started * versions:

[Freeciv-Dev] [bug #15502] Error message Illegal treaty ... they already have this treaty level

2010-07-11 Thread Daniel Markstedt
Update of bug #15502 (project freeciv): Planned Release: 2.2.2 = ___ Reply to this item at: http://gna.org/bugs/?15502 ___ Message sent

[Freeciv-Dev] [bug #15461] No horizontal scrollbar in Inspect City report

2010-07-11 Thread Daniel Markstedt
Update of bug #15461 (project freeciv): Planned Release: 2.2.2 = ___ Reply to this item at: http://gna.org/bugs/?15461 ___ Message sent

[Freeciv-Dev] [bug #16142] Some sources not included in tarball

2010-07-11 Thread Daniel Markstedt
Update of bug #16142 (project freeciv): Planned Release: 2.2.2 = ___ Reply to this item at: http://gna.org/bugs/?16142 ___ Message sent

[Freeciv-Dev] [patch #1698] Dahomey

2010-07-11 Thread Daniel Markstedt
Update of patch #1698 (project freeciv): Assigned to:None = mixcoatl ___ Follow-up Comment #3: Drew a couple of new heads with a cruder look. Put two of them in 'bowl' as well. (file