[Freeciv-Dev] [bug #19447] Over maximum veteran level assigned initially to unit
Follow-up Comment #8, bug #19447 (project freeciv): I agree that (2) should be safe. FWIW, I think the unused veteran level fields in RULESET_UNIT packets are initialised to zeroes, by virtue of the implicit zero initialisation of unittype.c:unit_type[] on the server. ___ Reply to this item at: http://gna.org/bugs/?19447 ___ Message sent via/by Gna! http://gna.org/ ___ Freeciv-dev mailing list Freeciv-dev@gna.org https://mail.gna.org/listinfo/freeciv-dev
[Freeciv-Dev] [patch #3141] Remove statically initialized roads information completely
Follow-up Comment #2, patch #3141 (project freeciv): - road_by_special() returns NULL if special in question is S_LAST - Ruleset loading gives error if compatibility special None is given as savegame handling code does not support it yet. (file #15197) ___ Additional Item Attachment: File name: NoStaticRoads-3.diff Size:11 KB ___ Reply to this item at: http://gna.org/patch/?3141 ___ Message sent via/by Gna! http://gna.org/ ___ Freeciv-dev mailing list Freeciv-dev@gna.org https://mail.gna.org/listinfo/freeciv-dev
[Freeciv-Dev] [bug #19472] Wrong activity icon for ACTIVITY_GEN_ROAD
Update of bug #19472 (project freeciv): Status: Ready For Test = Fixed Assigned to:None = cazfi Open/Closed:Open = Closed ___ Reply to this item at: http://gna.org/bugs/?19472 ___ Message sent via/by Gna! http://gna.org/ ___ Freeciv-dev mailing list Freeciv-dev@gna.org https://mail.gna.org/listinfo/freeciv-dev
[Freeciv-Dev] [patch #3130] Random disasters module
Update of patch #3130 (project freeciv): Status: Ready For Test = Done Assigned to:None = cazfi Open/Closed:Open = Closed ___ Reply to this item at: http://gna.org/patch/?3130 ___ Message sent via/by Gna! http://gna.org/ ___ Freeciv-dev mailing list Freeciv-dev@gna.org https://mail.gna.org/listinfo/freeciv-dev
[Freeciv-Dev] [bug #19467] Ignored requirement ranges for effect Unit_Recover
Update of bug #19467 (project freeciv): Status: Ready For Test = Fixed Assigned to:None = cazfi Open/Closed:Open = Closed ___ Reply to this item at: http://gna.org/bugs/?19467 ___ Message sent via/by Gna! http://gna.org/ ___ Freeciv-dev mailing list Freeciv-dev@gna.org https://mail.gna.org/listinfo/freeciv-dev
[Freeciv-Dev] [patch #3132] Default ruleset description to make clear that it's mainly meant for single player
Update of patch #3132 (project freeciv): Status: Ready For Test = Done Assigned to:None = cazfi Open/Closed:Open = Closed ___ Reply to this item at: http://gna.org/patch/?3132 ___ Message sent via/by Gna! http://gna.org/ ___ Freeciv-dev mailing list Freeciv-dev@gna.org https://mail.gna.org/listinfo/freeciv-dev
[Freeciv-Dev] [bug #19487] Tech loss in ruleset without tech upkeep
Follow-up Comment #1, bug #19487 (project freeciv): Attached patch simply restores old behavior of never losing tech when ruleset has no tech upkeep. We still want better fix for tech loss rules for tech upkeep case, in trunk at least, but until then this fix already makes rulesets without tech upkeep playble. (file #15198) ___ Additional Item Attachment: File name: NoTechLossInDefaultRules.diff Size:0 KB ___ Reply to this item at: http://gna.org/bugs/?19487 ___ Message sent via/by Gna! http://gna.org/ ___ Freeciv-dev mailing list Freeciv-dev@gna.org https://mail.gna.org/listinfo/freeciv-dev
[Freeciv-Dev] [bug #19447] Over maximum veteran level assigned initially to unit
Update of bug #19447 (project freeciv): Status: In Progress = Ready For Test ___ Follow-up Comment #9: 2) Change client u-veteran_levels = 0 to u-veteran_levels = MAX_VET_LEVELS Untested patch to do just that. (file #15199) ___ Additional Item Attachment: File name: MaxVetLvlToMax.diffSize:0 KB ___ Reply to this item at: http://gna.org/bugs/?19447 ___ Message sent via/by Gna! http://gna.org/ ___ Freeciv-dev mailing list Freeciv-dev@gna.org https://mail.gna.org/listinfo/freeciv-dev
[Freeciv-Dev] [bug #19487] Tech loss in ruleset without tech upkeep
Follow-up Comment #2, bug #19487 (project freeciv): Another case to consider, this time with tech upkeep enabled: Player research technology that enables new government. Number of bulbs carried over is happens to be very low. Player immediately issues revolution, entering Anarchy during which bulbs are not produced. Tech upkeep takes all his bulbs and triggers tech loss. As number of bulbs is restored to value 0 only, from which tech upkeep then reduces it to negative value, new tech is lost every turn during the anarchy. One of these techs is probably one that enabled new government, making it impossible to switch to new government in the end. ___ Reply to this item at: http://gna.org/bugs/?19487 ___ Message sent via/by Gna! http://gna.org/ ___ Freeciv-dev mailing list Freeciv-dev@gna.org https://mail.gna.org/listinfo/freeciv-dev
[Freeciv-Dev] [bug #19488] Gtk3-client packaging still expects binary to be named freeciv-gtk2x
URL: http://gna.org/bugs/?19488 Summary: Gtk3-client packaging still expects binary to be named freeciv-gtk2x Project: Freeciv Submitted by: cazfi Submitted on: Thu 23 Feb 2012 12:25:06 AM EET Category: general Severity: 3 - Normal Priority: 5 - Normal Status: Ready For Test Assigned to: None Originator Email: Open/Closed: Open Release: Discussion Lock: Any Operating System: None Planned Release: 2.4.0, 2.5.0 ___ Details: Debian packaging still refers to temporary development name of gtk3-client. Fix attached. ___ File Attachments: --- Date: Thu 23 Feb 2012 12:25:06 AM EET Name: DebGtk3.diff Size: 1016B By: cazfi http://gna.org/bugs/download.php?file_id=15200 ___ Reply to this item at: http://gna.org/bugs/?19488 ___ Message sent via/by Gna! http://gna.org/ ___ Freeciv-dev mailing list Freeciv-dev@gna.org https://mail.gna.org/listinfo/freeciv-dev
[Freeciv-Dev] [bug #19029] Possible trouble saving/loading players' known territory for player indices =40
Update of bug #19029 (project freeciv): Status:None = Ready For Test ___ Follow-up Comment #8: Untested fix for S2_3 and S2_4. In those branches we have to save in a format that also earlier versions can load. In trunk we could (and eventually should) drop saving in old bit order. But we cannot commit this one to stable branches without trunk having at least same changes making it possible to load savegames from stable branches. Dropping saving in old bit order in trunk can wait for a new ticket, and this one should go in after testing in all three branches. (file #15201) ___ Additional Item Attachment: File name: KnownSaveBitOrder-S2_3.diffSize:3 KB ___ Reply to this item at: http://gna.org/bugs/?19029 ___ Message sent via/by Gna! http://gna.org/ ___ Freeciv-dev mailing list Freeciv-dev@gna.org https://mail.gna.org/listinfo/freeciv-dev
[Freeciv-Dev] [patch #3154] Remove road removal code when city destroyed
URL: http://gna.org/patch/?3154 Summary: Remove road removal code when city destroyed Project: Freeciv Submitted by: cazfi Submitted on: Thu 23 Feb 2012 01:16:49 AM EET Category: general Priority: 5 - Normal Status: Ready For Test Privacy: Public Assigned to: None Originator Email: Open/Closed: Open Discussion Lock: Any Planned Release: 2.5.0 ___ Details: There's old code remaining from time when city centers always had roads that removes roads when city is removed if terrain cannot support roads. It's not only unnecessary now that road would not be present in such a city center anyway, but code in question handles roads old way as specials. Attached patch simply removes the code. ___ File Attachments: --- Date: Thu 23 Feb 2012 01:16:49 AM EET Name: ObsRoadRemWhenCityRem.diff Size: 587B By: cazfi http://gna.org/patch/download.php?file_id=15202 ___ Reply to this item at: http://gna.org/patch/?3154 ___ Message sent via/by Gna! http://gna.org/ ___ Freeciv-dev mailing list Freeciv-dev@gna.org https://mail.gna.org/listinfo/freeciv-dev
[Freeciv-Dev] [patch #3155] Remove overzealous sanity check about Road under Railroad
URL: http://gna.org/patch/?3155 Summary: Remove overzealous sanity check about Road under Railroad Project: Freeciv Submitted by: cazfi Submitted on: Thu 23 Feb 2012 01:20:49 AM EET Category: general Priority: 5 - Normal Status: Ready For Test Privacy: Public Assigned to: None Originator Email: Open/Closed: Open Discussion Lock: Any Planned Release: 2.5.0 ___ Details: Now that reqauirements of road types are not hard-coded but read from ruleset, sanity check that checks road to exist in any tile that has railroad can give false errors on some rulesets. Attached patch removes the sanity check. ___ File Attachments: --- Date: Thu 23 Feb 2012 01:20:49 AM EET Name: IndepRailSanity.diff Size: 745B By: cazfi http://gna.org/patch/download.php?file_id=15203 ___ Reply to this item at: http://gna.org/patch/?3155 ___ Message sent via/by Gna! http://gna.org/ ___ Freeciv-dev mailing list Freeciv-dev@gna.org https://mail.gna.org/listinfo/freeciv-dev
[Freeciv-Dev] [patch #3141] Remove statically initialized roads information completely
Follow-up Comment #3, patch #3141 (project freeciv): - Send compat_special to client side (file #15204) ___ Additional Item Attachment: File name: NoStaticRoads-4.diff Size:13 KB ___ Reply to this item at: http://gna.org/patch/?3141 ___ Message sent via/by Gna! http://gna.org/ ___ Freeciv-dev mailing list Freeciv-dev@gna.org https://mail.gna.org/listinfo/freeciv-dev
[Freeciv-Dev] [bug #19490] can_unit_do_activity_road() checks wrong activity
URL: http://gna.org/bugs/?19490 Summary: can_unit_do_activity_road() checks wrong activity Project: Freeciv Submitted by: cazfi Submitted on: Thu 23 Feb 2012 03:21:37 AM EET Category: general Severity: 3 - Normal Priority: 5 - Normal Status: Ready For Test Assigned to: None Originator Email: Open/Closed: Open Release: Discussion Lock: Any Operating System: None Planned Release: 2.5.0 ___ Details: can_unit_do_activity_road() checks if unit can do ACTIVITY_ROAD instead of checking ACTIVITY_GEN_ROAD. Fix attached ___ File Attachments: --- Date: Thu 23 Feb 2012 03:21:37 AM EET Name: CanUnitActGenRoad.diff Size: 516B By: cazfi http://gna.org/bugs/download.php?file_id=15205 ___ Reply to this item at: http://gna.org/bugs/?19490 ___ Message sent via/by Gna! http://gna.org/ ___ Freeciv-dev mailing list Freeciv-dev@gna.org https://mail.gna.org/listinfo/freeciv-dev
[Freeciv-Dev] [patch #3156] Fix remaining comments referring to client/civclient.c
URL: http://gna.org/patch/?3156 Summary: Fix remaining comments referring to client/civclient.c Project: Freeciv Submitted by: cazfi Submitted on: Thu 23 Feb 2012 07:31:16 AM EET Category: client Priority: 5 - Normal Status: Ready For Test Privacy: Public Assigned to: None Originator Email: Open/Closed: Open Discussion Lock: Any Planned Release: 2.5.0 ___ Details: A couple of comments still mention client/civclient.c where client/client_main.c is meant. Attached patch changes those. ___ File Attachments: --- Date: Thu 23 Feb 2012 07:31:16 AM EET Name: CivclientCComments.diff Size: 1kB By: cazfi http://gna.org/patch/download.php?file_id=15206 ___ Reply to this item at: http://gna.org/patch/?3156 ___ Message sent via/by Gna! http://gna.org/ ___ Freeciv-dev mailing list Freeciv-dev@gna.org https://mail.gna.org/listinfo/freeciv-dev
[Freeciv-Dev] [patch #3150] Trait aggressive
Follow-up Comment #1, patch #3150 (project freeciv): - Added commented out trait.aggressive entry to default nations.ruleset (file #15207) ___ Additional Item Attachment: File name: TraitAggressive-2.diff Size:3 KB ___ Reply to this item at: http://gna.org/patch/?3150 ___ Message sent via/by Gna! http://gna.org/ ___ Freeciv-dev mailing list Freeciv-dev@gna.org https://mail.gna.org/listinfo/freeciv-dev
[Freeciv-Dev] [bug #19458] assertion 'activity_requires_target(new_activity)' failed.
Update of bug #19458 (project freeciv): Status: Ready For Test = Fixed Assigned to:None = cazfi Open/Closed:Open = Closed ___ Reply to this item at: http://gna.org/bugs/?19458 ___ Message sent via/by Gna! http://gna.org/ ___ Freeciv-dev mailing list Freeciv-dev@gna.org https://mail.gna.org/listinfo/freeciv-dev
[Freeciv-Dev] [patch #3145] Remove client/civclient.c from version control
Update of patch #3145 (project freeciv): Status:None = Done Assigned to:None = cazfi Open/Closed:Open = Closed ___ Reply to this item at: http://gna.org/patch/?3145 ___ Message sent via/by Gna! http://gna.org/ ___ Freeciv-dev mailing list Freeciv-dev@gna.org https://mail.gna.org/listinfo/freeciv-dev
[Freeciv-Dev] [bug #19467] Ignored requirement ranges for effect Unit_Recover
Update of bug #19467 (project freeciv): Status: Fixed = In Progress Open/Closed: Closed = Open ___ Follow-up Comment #4: Autogame is now spitting high number of failed assserts in get_unittype_bonus(), most likely because of this commit. ___ Reply to this item at: http://gna.org/bugs/?19467 ___ Message sent via/by Gna! http://gna.org/ ___ Freeciv-dev mailing list Freeciv-dev@gna.org https://mail.gna.org/listinfo/freeciv-dev