[Freeciv-Dev] [patch #3815] Provide information about unit class nativity in terrain help

2013-04-09 Thread Emmet Hikory
Follow-up Comment #5, patch #3815 (project freeciv): The attached files provide one possible adjustment to the static helptexts for S2_4 and trunk. In trunk, I did not remove the phrases "Planet's radiation makes it impossible for Earthly organisms to survive here.", "Burrowing units are unable t

[Freeciv-Dev] [bug #20698] Version check incorrectly considers "2.4.0" older than "2.4.0-beta1"

2013-04-09 Thread Jacob Nevins
Update of bug #20698 (project freeciv): Status: Ready For Test => Fixed Open/Closed:Open => Closed ___ Reply to this item at:

[Freeciv-Dev] [patch #3323] Allow requirement range "within city workable radius" for tile-based requirements?

2013-04-09 Thread Jacob Nevins
Update of patch #3323 (project freeciv): Planned Release: => 2.5.0 ___ Follow-up Comment #7: Looks like the original intent was addressed by patch #3740. There are some other, more compli

[Freeciv-Dev] [patch #3740] Support "City" range requirements for tile properties

2013-04-09 Thread Jacob Nevins
Follow-up Comment #2, patch #3740 (project freeciv): See also patch #3323. ___ Reply to this item at: ___ Message sent via/by Gna! http://gna.org/ __

[Freeciv-Dev] Can I un-static function?

2013-04-09 Thread M
I made scalable minimap for qt-client ( default is integer only, that one is fully scalable), but I had to draw viewport manually. For that I had to convert gui to overview position, but that function is static ( in overview_common.c), so I used gui to map conversion and then map to overview (

[Freeciv-Dev] [patch #3843] Donate citizens of own nationality to allies' cities

2013-04-09 Thread Jacob Nevins
URL: Summary: Donate citizens of own nationality to allies' cities Project: Freeciv Submitted by: jtn Submitted on: Tue Apr 9 21:50:01 2013 Category: general Priority: 5 - Norm

[Freeciv-Dev] [patch #2265] [metaticket] citizen nationality

2013-04-09 Thread Jacob Nevins
Update of patch #2265 (project freeciv): Status: In Progress => Done Open/Closed:Open => Closed ___ Follow-up Comment #13: OK, it seems this

[Freeciv-Dev] [patch #2265] [metaticket] citizen nationality

2013-04-09 Thread Jacob Nevins
Update of patch #2265 (project freeciv): Dependency Removed: => patch #2366 ___ Reply to this item at: ___ Message sen

[Freeciv-Dev] [patch #2265] [metaticket] citizen nationality

2013-04-09 Thread Jacob Nevins
Update of patch #2265 (project freeciv): Dependency Removed: => patch #2367 ___ Reply to this item at: ___ Message sen

[Freeciv-Dev] [patch #2367] [citizens13] poison citizens of the city owner

2013-04-09 Thread Jacob Nevins
Update of patch #2367 (project freeciv): Planned Release: 2.4.0 => 2.5.0 ___ Follow-up Comment #6: Don't think this is going to make 2.4.0. _

[Freeciv-Dev] [patch #2367] [citizens13] poison citizens of the city owner

2013-04-09 Thread Jacob Nevins
Follow-up Comment #7, patch #2367 (project freeciv): (in that from discussion I think it's going to want to be optional and need effects and what have you, so will break ruleset format freeze) ___ Reply to this item at:

[Freeciv-Dev] [bug #17680] assertion 'vsystem->levels > level' failed

2013-04-09 Thread Jacob Nevins
Update of bug #17680 (project freeciv): Status:None => Works For Me Open/Closed:Open => Closed ___ Follow-up Comment #6: Well, we didn't hav

[Freeciv-Dev] [patch #3815] Provide information about unit class nativity in terrain help

2013-04-09 Thread Marko Lindqvist
Follow-up Comment #8, patch #3815 (project freeciv): I committed the helptext generation part, but would really like more opinions of the changes to static texts. Old description is redundant, but is it still something that makes people to realize the difference between Ocean and Deep Ocean while

[Freeciv-Dev] [bug #20731] startpos_iter_init(): warning: type-punning to incomplete type might break strict-aliasing rules

2013-04-09 Thread Jacob Nevins
URL: Summary: startpos_iter_init(): warning: type-punning to incomplete type might break strict-aliasing rules Project: Freeciv Submitted by: jtn Submitted on: Tue Apr 9 22:23:00 2013 Category

[Freeciv-Dev] [patch #3805] Allow paratroopers to land on transport

2013-04-09 Thread Marko Lindqvist
Update of patch #3805 (project freeciv): Status: Ready For Test => Done Open/Closed:Open => Closed ___ Reply to this item at:

[Freeciv-Dev] [bug #20666] Disaster flood requires river special

2013-04-09 Thread Marko Lindqvist
Update of bug #20666 (project freeciv): Status: Ready For Test => Fixed Open/Closed:Open => Closed ___ Reply to this item at:

[Freeciv-Dev] [patch #3836] Documentation update for requirements

2013-04-09 Thread Marko Lindqvist
Update of patch #3836 (project freeciv): Status: Ready For Test => Done Open/Closed:Open => Closed ___ Reply to this item at:

[Freeciv-Dev] [patch #3835] Negated requirements sanity checking improvements

2013-04-09 Thread Marko Lindqvist
Follow-up Comment #9, patch #3835 (project freeciv): Reading the patch once more, I noticed that for requirements to be considered opposites they should have same 'survives'. I'm 95% sure 'survives' shouldn't matter here. Of course, 5% is still a big uncertainty. _

[Freeciv-Dev] [bug #20507] Barbarian nations in unknown group in civ2 ruleset

2013-04-09 Thread Jacob Nevins
Update of bug #20507 (project freeciv): Status:None => Ready For Test Assigned to:None => jtn ___ Follow-up Comment #3: Attached patch copi

[Freeciv-Dev] [bug #20626] Client crashes with GLib-GObject-CRITICAL **: g_object_unref: assertion `G_IS_OBJECT (object)' failed

2013-04-09 Thread Jacob Nevins
Follow-up Comment #2, bug #20626 (project freeciv): When I run with -d 3 the server's last words are ... 3: sending packet type=PACKET_UNIT_SHORT_INFO(64) len=8 to jtn 3: attack:20, defense:112, attack firepower:1, defense firepower:1 3: Teleported Pirate Barbarian Leader from (209,103) to Garbów

[Freeciv-Dev] [bug #20626] Client crashes with GLib-GObject-CRITICAL **: g_object_unref: assertion `G_IS_OBJECT (object)' failed

2013-04-09 Thread Marko Lindqvist
Follow-up Comment #3, bug #20626 (project freeciv): Being about transport destruction and Barbarian Leader teleporting away, this sounds exactly like bug #20699 - except that that was introduced to S2_4 / TRUNK only and never existed in S2_3. Maybe there is something a bit alike in S2_3 implementa

[Freeciv-Dev] [bug #20626] Client crashes with GLib-GObject-CRITICAL **: g_object_unref: assertion `G_IS_OBJECT (object)' failed

2013-04-09 Thread Jacob Nevins
Follow-up Comment #4, bug #20626 (project freeciv): Here's a backtrace with head-of-S2_3 (r22707) compiled with -O0 -g. #0 0x00427532 in server_remove_unit (punit=0x3764af0) at unittools.c:1434 pcargo_iter = 0x1f pcargo = 0x6c657661726143 ptile = 0x7f2add0cf0

[Freeciv-Dev] [patch #3835] Negated requirements sanity checking improvements

2013-04-09 Thread Emmet Hikory
Follow-up Comment #10, patch #3835 (project freeciv): I'm inclined to agree that survives shouldn't matter (the code was written to compare the code structures, rather than with careful semantic analysis). That said, I can think of a couple cases where one might use this: a) Creating a condi

[Freeciv-Dev] [patch #3804] Enabling restricted air transport

2013-04-09 Thread Emmet Hikory
Follow-up Comment #20, patch #3804 (project freeciv): Update to the experimental ruleset patch to take into account changes in README.ruleset_experimental after revision 22705. (file #17733) ___ Additional Item Attachment: File name: exper

[Freeciv-Dev] [bug #20626] Client crashes with GLib-GObject-CRITICAL **: g_object_unref: assertion `G_IS_OBJECT (object)' failed

2013-04-09 Thread Jacob Nevins
Follow-up Comment #5, bug #20626 (project freeciv): Still not sure what's going on, but some facts * Sunk boat is a Caravel belonging to Blackbeard (Pirate) ** Was sunk by Dravya Shah (Nepali) * At [l tgt="tile" x=209 y=103 /] * Three ex-cargo at time of crash (IDs 1785, 1786, 1787 -- looking at p