[Freeciv-Dev] [bug #21346] Behaviour on saving without have_resources poorly defined

2013-12-14 Thread Jacob Nevins
URL: http://gna.org/bugs/?21346 Summary: Behaviour on saving without have_resources poorly defined Project: Freeciv Submitted by: jtn Submitted on: Sat Dec 14 13:26:42 2013 Category: None Severity: 3 -

[Freeciv-Dev] [bug #21347] Ability to not save resources not exposed to scenario editor

2013-12-14 Thread Jacob Nevins
Update of bug #21347 (project freeciv): Depends on: = bugs #21346 ___ Reply to this item at: http://gna.org/bugs/?21347 ___ Message sent

[Freeciv-Dev] [bug #21347] Ability to not save resources not exposed to scenario editor

2013-12-14 Thread Jacob Nevins
URL: http://gna.org/bugs/?21347 Summary: Ability to not save resources not exposed to scenario editor Project: Freeciv Submitted by: jtn Submitted on: Sat Dec 14 13:27:03 2013 Category: None Severity: 3 -

[Freeciv-Dev] [bug #21346] Behaviour on saving without have_resources poorly defined

2013-12-14 Thread Marko Lindqvist
Follow-up Comment #1, bug #21346 (project freeciv): (There's also a rivers_overlay wart in saving, but since it I don't think server ever *saves* as rivers_overlay, it just loads (old) scenarios where it's set. What is missing here, is having savegame format support for such scenarios with

[Freeciv-Dev] [bug #21348] server/edithand.c:handle_scenario_info() looks redundant

2013-12-14 Thread Jacob Nevins
URL: http://gna.org/bugs/?21348 Summary: server/edithand.c:handle_scenario_info() looks redundant Project: Freeciv Submitted by: jtn Submitted on: Sat Dec 14 15:48:30 2013 Category: None Severity: 2 -

[Freeciv-Dev] [bug #21349] Handicaps pointer rather than contents copied on civil war = double free()

2013-12-14 Thread Jacob Nevins
URL: http://gna.org/bugs/?21349 Summary: Handicaps pointer rather than contents copied on civil war = double free() Project: Freeciv Submitted by: jtn Submitted on: Sat Dec 14 16:03:50 2013 Category: ai

[Freeciv-Dev] [bug #21349] Handicaps pointer rather than contents copied on civil war = double free()

2013-12-14 Thread Jacob Nevins
Update of bug #21349 (project freeciv): Status: In Progress = Ready For Test ___ Additional Item Attachment: File name: trunk-handicaps-copy.patch Size:2 KB

[Freeciv-Dev] [bug #21349] Handicaps pointer rather than contents copied on civil war = double free()

2013-12-14 Thread Marko Lindqvist
Follow-up Comment #1, bug #21349 (project freeciv): Why are handicaps copied there at all? Shouldn't it be function of ai difficulty level? Given that difficulty level of the created player can be different from original one, this even sounds like a bug.

[Freeciv-Dev] [bug #21349] Handicaps pointer rather than contents copied on civil war = double free()

2013-12-14 Thread Jacob Nevins
Update of bug #21349 (project freeciv): Status: Ready For Test = None Assigned to: jtn = None ___ Follow-up Comment #2: Hm, I wasn't really

[Freeciv-Dev] [bug #21349] Handicaps pointer rather than contents copied on civil war = double free()

2013-12-14 Thread Marko Lindqvist
Follow-up Comment #3, bug #21349 (project freeciv): Indeed, this wouldn't be appropriate if you wanted a different AI type for civil-war players. Not only if you want different AI type (actually that does not make difference at the moment as handicap setting is not yet AI type dependant), but

[Freeciv-Dev] [bug #21268] Scenario start positions can't restrict nation choice for players

2013-12-14 Thread Jacob Nevins
Update of bug #21268 (project freeciv): Status:None = Ready For Test Assigned to:None = jtn Planned Release: = 2.5.0,2.6.0 Summary:

[Freeciv-Dev] [bug #21268] Scenario start positions can't restrict nation choice for players

2013-12-14 Thread Jacob Nevins
Follow-up Comment #3, bug #21268 (project freeciv): (The label for this in the editor, Nation Start Positions, leaves a lot to be desired; unfortunately it needs to be short. Should really dust off pepeto's editor tooltips patch from bug #15580.)

[Freeciv-Dev] [bug #21349] Handicaps pointer rather than contents copied on civil war = double free()

2013-12-14 Thread Jacob Nevins
Follow-up Comment #4, bug #21349 (project freeciv): I have not tested, but I think it's as simple as leaving that line out completely as call to set_ai_level_direct() a couple of lines later should do the right thing Looks plausible (but I haven't tested either)... (and yes, actually it

[Freeciv-Dev] [patch #4337] Worker thread for freeciv-mp-qt

2013-12-14 Thread Marko Lindqvist
URL: http://gna.org/patch/?4337 Summary: Worker thread for freeciv-mp-qt Project: Freeciv Submitted by: cazfi Submitted on: Sat 14 Dec 2013 11:44:47 PM EET Category: module installer Priority: 5 - Normal

[Freeciv-Dev] [patch #4329] Reimplement special name functions to savecompat

2013-12-14 Thread Marko Lindqvist
Update of patch #4329 (project freeciv): Status: Ready For Test = Done Assigned to:None = cazfi Open/Closed:Open = Closed