Re: [Freeciv-Dev] Gna email problems

2009-07-05 Thread Pepeto
2009/7/4 cazfi wrote: It seems that commit messages are not being sent to freeciv-commits. Neither does cia.vc see them. http://cia.vc/stats/project/FreeCiv - ML I already experienced this problem when I started to work in warclient. It is actually due to the sebnding rights. I think Book

[Freeciv-Dev] [bug #13865] iterator.h doesn't include the definition of type 'bool'

2009-07-05 Thread pepeto
URL: http://gna.org/bugs/?13865 Summary: iterator.h doesn't include the definition of type 'bool' Project: Freeciv Submitted by: pepeto Submitted on: Sunday 07/05/2009 at 07:22 Category: general Severity:

[Freeciv-Dev] (PR#40841) Bug: Corrupted city name crashes Freeciv

2009-07-05 Thread Paul Angel
URL: http://bugs.freeciv.org/Ticket/Display.html?id=40841 Dear people, I'm playing Freeciv-2.1.9-win32-gtk2 on Windows XP. I'm playing the Catalans on normal difficulty - it's now 1140 AD. The last few sessions, when I build a new city, the name that comes up first is Perpiny - with an X in

Re: [Freeciv-Dev] [bug #13848] [Patch] More detailed coding style guidelines

2009-07-05 Thread Per Inge Mathisen
While on the topic of asserts, I recommend using a macro similar to the ASSERT_OR_RETURN that I wrote for Warzone2100 -- http://warzone2100.svn.sourceforge.net/viewvc/warzone2100/trunk/lib/framework/debug.h?revision=7821view=markup The reason for this is that 1) you avoid having to do such double

[Freeciv-Dev] Tracking changes to translatable strings

2009-07-05 Thread Marko Lindqvist
We are nearing S2_2 branching point and first beta in a couple of months. This means also that new and changes translatable strings should be made available to translators faster. I have run update-po for translations every couple of months, but now, until S2_2 branching, I would want to run

[Freeciv-Dev] [bug #13846] recursion checking for the read command

2009-07-05 Thread Matthias Pfafferodt
Follow-up Comment #6, bug #13846 (project freeciv): updated patch changes: - max 10 calls to read - check is possible for the read command (used for voting?) (file #6132) ___ Additional Item Attachment: File name:

[Freeciv-Dev] [bug #13843] [Patch] Build base menu

2009-07-05 Thread Marko Lindqvist
Update of bug #13843 (project freeciv): Status:None = Fixed Assigned to:None = cazfi Open/Closed:Open = Closed

[Freeciv-Dev] [bug #13841] [Patch] Worklist clean up

2009-07-05 Thread Marko Lindqvist
Follow-up Comment #1, bug #13841 (project freeciv): Please try to split your patches to smaller separate tickets which each do just one thing. That way it's much easier to confirm them. ___ Reply to this item at:

[Freeciv-Dev] [bug #13867] [patch 01/07] get game settings via wrapper functions

2009-07-05 Thread Matthias Pfafferodt
URL: http://gna.org/bugs/?13867 Summary: [patch 01/07] get game settings via wrapper functions Project: Freeciv Submitted by: syntron Submitted on: Sonntag 05.07.2009 um 22:15 Category: general Severity:

[Freeciv-Dev] [bug #13868] [patch 02/07] use an union in the struct setting

2009-07-05 Thread Matthias Pfafferodt
URL: http://gna.org/bugs/?13868 Summary: [patch 02/07] use an union in the struct setting Project: Freeciv Submitted by: syntron Submitted on: Sonntag 05.07.2009 um 22:15 Category: general Severity: 2 -

[Freeciv-Dev] [bug #13869] [patch 03/07] simplify validation functions

2009-07-05 Thread Matthias Pfafferodt
URL: http://gna.org/bugs/?13869 Summary: [patch 03/07] simplify validation functions Project: Freeciv Submitted by: syntron Submitted on: Sonntag 05.07.2009 um 22:16 Category: general Severity: 2 - Minor

[Freeciv-Dev] [bug #13870] [patch 04/07] extra functions to check and update settings

2009-07-05 Thread Matthias Pfafferodt
URL: http://gna.org/bugs/?13870 Summary: [patch 04/07] extra functions to check and update settings Project: Freeciv Submitted by: syntron Submitted on: Sonntag 05.07.2009 um 22:17 Category: general

[Freeciv-Dev] [bug #13850] add min/max definitions for saveturns

2009-07-05 Thread Marko Lindqvist
Update of bug #13850 (project freeciv): Status:None = Fixed Open/Closed:Open = Closed ___ Reply to this item at:

[Freeciv-Dev] [bug #13846] recursion checking for the read command

2009-07-05 Thread Marko Lindqvist
Update of bug #13846 (project freeciv): Assigned to:None = cazfi ___ Reply to this item at: http://gna.org/bugs/?13846 ___ Message sent

[Freeciv-Dev] [bug #13841] [Patch] Worklist clean up

2009-07-05 Thread pepeto
Follow-up Comment #2, bug #13841 (project freeciv): I'm not sure I could make separated patches for those changes because all of them are linked. ___ Reply to this item at: http://gna.org/bugs/?13841

[Freeciv-Dev] [bug #13841] [Patch] Worklist clean up

2009-07-05 Thread Marko Lindqvist
Follow-up Comment #3, bug #13841 (project freeciv): I try to make more detailed splitting proposal when I have time to look this patch in more detail (or maybe I'll end admitting that it's not splittable :-) Anyway, obvious first patch would be init_worklist() - worklist_init(), copy_worklist()

[Freeciv-Dev] [bug #13863] Remove old server goto code, step 1: warmap alternatives

2009-07-05 Thread Marko Lindqvist
Follow-up Comment #1, bug #13863 (project freeciv): There is a minor change in the pf code itself. I noticed that the node could be initialized many times if the first time, the node was considered unreachable. I added a new node status NS_INIT which just marks to don't initialize the node a

[Freeciv-Dev] [bug #13872] [patch 06/07] move sset_is_to_client() to settings.c

2009-07-05 Thread Matthias Pfafferodt
URL: http://gna.org/bugs/?13872 Summary: [patch 06/07] move sset_is_to_client() to settings.c Project: Freeciv Submitted by: syntron Submitted on: Sonntag 05.07.2009 um 22:19 Category: general Severity: 2

[Freeciv-Dev] [bug #13873] [patch 07/07] load game settings from ruleset file

2009-07-05 Thread Matthias Pfafferodt
URL: http://gna.org/bugs/?13873 Summary: [patch 07/07] load game settings from ruleset file Project: Freeciv Submitted by: syntron Submitted on: Sonntag 05.07.2009 um 22:21 Category: general Severity: 2 -

[Freeciv-Dev] [bug #13875] change O_MAX and O_COUNT to O_LAST

2009-07-05 Thread Matthias Pfafferodt
URL: http://gna.org/bugs/?13875 Summary: change O_MAX and O_COUNT to O_LAST Project: Freeciv Submitted by: syntron Submitted on: Sonntag 05.07.2009 um 23:06 Category: general Severity: 2 - Minor

[Freeciv-Dev] [bug #13875] change O_MAX and O_COUNT to O_LAST

2009-07-05 Thread Marko Lindqvist
Update of bug #13875 (project freeciv): Assigned to:None = cazfi ___ Reply to this item at: http://gna.org/bugs/?13875 ___ Message sent

[Freeciv-Dev] [bug #13843] [Patch] Build base menu

2009-07-05 Thread John Keller
Follow-up Comment #1, bug #13843 (project freeciv): I'm sorry I'm posting after this bug is closed, but it's of interest to me. I'm in the middle of redoing a menu layout change that I originally submitted about a year ago. Would you be able to go into a bit more depth about why this path moves