[Freeciv-Dev] Kampuchean.ruleset

2007-01-10 Thread Marko Lindqvist
Is there some reason why empty kampuchean.ruleset still exist in svn? Real ruleset is cambodian.ruleset. - ML ___ Freeciv-dev mailing list Freeciv-dev@gna.org https://mail.gna.org/listinfo/freeciv-dev

Re: [Freeciv-Dev] Re: (PR#18773) Reordering worklist smeshes it up if there is just bought unit

2007-01-11 Thread Marko Lindqvist
URL: http://bugs.freeciv.org/Ticket/Display.html?id=18773 On 7/21/06, Jason Short [EMAIL PROTECTED] wrote: Alternately the client just needs to make all the same checks the server does, BEFORE sending the packets. That's what Tommi's patch does. I'm going to commit it soon if nobody

[Freeciv-Dev] (PR#33324) [Patch] Hardcoded City Walls cost in AI code

2007-01-11 Thread Marko Lindqvist
URL: http://bugs.freeciv.org/Ticket/Display.html?id=33324 This patch replaces hardcoded city walls cost with call to impr_buy_gold_cost(). Note that hardcoded value were no longer true even for default ruleset. - ML diff -Nurd -X.diff_ignore freeciv/ai/advmilitary.c

[Freeciv-Dev] (PR#33343) S2_1 singaporean.ruleset broken

2007-01-11 Thread Marko Lindqvist
URL: http://bugs.freeciv.org/Ticket/Display.html?id=33343 0: Nation Singaporean: the leader sex count (5) is not equal to the number of leaders (4) - ML ___ Freeciv-dev mailing list Freeciv-dev@gna.org https://mail.gna.org/listinfo/freeciv-dev

[Freeciv-Dev] (PR#11068) savegame file compression

2007-01-12 Thread Marko Lindqvist
URL: http://bugs.freeciv.org/Ticket/Display.html?id=11068 Patch updated against svn. Second patch is for disabling bzip2 saves in pubserver. which cannot handle them. - ML diff -Nurd -X.diff_ignore freeciv/client/connectdlg_common.c freeciv/client/connectdlg_common.c ---

Re: [Freeciv-Dev] (PR#21843) Freeciv 2.1.0 Beta2 bug.

2007-01-12 Thread Marko Lindqvist
URL: http://bugs.freeciv.org/Ticket/Display.html?id=21843 On 1/13/07, Marko Lindqvist [EMAIL PROTECTED] wrote: We have an alliance, but no shared vision. Alliance allows us to see units inside transport. When transport moves out of sight, transported units are not removed. Patch - ML

Re: [Freeciv-Dev] (PR#18476) [Patch] generate_warmap() and TerrainSpeed

2007-01-13 Thread Marko Lindqvist
URL: http://bugs.freeciv.org/Ticket/Display.html?id=18476 On 7/11/06, Marko Lindqvist [EMAIL PROTECTED] wrote: Make really_generate_warmap() UCF_TERRAIN_SPEED aware. I had to rewrite most of it. Patch updated. Take advantage of some new interfaces: - Use can_attack_non_native() instead

Re: [Freeciv-Dev] Re: (PR#19822) bug in path_finding.c

2007-01-13 Thread Marko Lindqvist
URL: http://bugs.freeciv.org/Ticket/Display.html?id=19822 1. First node always gets segment_length = 1. 2. No other node can get segment_length 1. 3. Segment_length can be too big, but not too small. This mostly untested patch modifies create_danger_segment() so that it uses segment_length

[Freeciv-Dev] (PR#14636) tech update bug

2007-01-13 Thread Marko Lindqvist
URL: http://bugs.freeciv.org/Ticket/Display.html?id=14636 Christian Knoke [EMAIL PROTECTED]: When an allied has stolen a tech form an enemy, the diplomacy dialog and intelligence is not updated. Same applies when Caravan gets initial science bonus from establishing trade route. Patch is

[Freeciv-Dev] (PR#16350) Beta1: Tutorial translation not possible

2007-01-13 Thread Marko Lindqvist
URL: http://bugs.freeciv.org/Ticket/Display.html?id=16350 This leads to the fact that the complete tutorial is not translatable. I believe this is because the translatable strings are marked as _('...') instead of _(...) I tried changing this which made gettext recognize the strings, but

Re: [Freeciv-Dev] (PR#16350) Beta1: Tutorial translation not possible

2007-01-13 Thread Marko Lindqvist
URL: http://bugs.freeciv.org/Ticket/Display.html?id=16350 S2_1 version of the tutorial patch. - ML diff -Nurd -X.diff_ignore freeciv/data/scenario/tutorial.sav freeciv/data/scenario/tutorial.sav --- freeciv/data/scenario/tutorial.sav 2006-07-17 14:28:11.0 +0300 +++

[Freeciv-Dev] (PR#33498) [Bug] Edit mode with two clients freezes

2007-01-13 Thread Marko Lindqvist
URL: http://bugs.freeciv.org/Ticket/Display.html?id=33498 Separately started server and two clients. Trying to select Editor - Edit mode from either client causes both of them to freeze. Chat window for both says 'Server set to edit mode' - 'Server leaves edit mode' over and over again.

Re: [Freeciv-Dev] (PR#9568) Bribing bombers

2007-01-13 Thread Marko Lindqvist
URL: http://bugs.freeciv.org/Ticket/Display.html?id=9568 On 1/13/07, Marko Lindqvist [EMAIL PROTECTED] wrote: Untested patch for carrying fuel and some other info over bribing. Moves_left should be already ok. Bomber probably attacked in the beginning of the next turn and not in the same

Re: [Freeciv-Dev] (PR#14636) tech update bug

2007-01-14 Thread Marko Lindqvist
URL: http://bugs.freeciv.org/Ticket/Display.html?id=14636 On 1/14/07, Marko Lindqvist [EMAIL PROTECTED] wrote: URL: http://bugs.freeciv.org/Ticket/Display.html?id=14636 Christian Knoke [EMAIL PROTECTED]: When an allied has stolen a tech form an enemy, the diplomacy dialog

Re: [Freeciv-Dev] (PR#18234) Feature request - Help dialog shows if units require a Building

2007-01-14 Thread Marko Lindqvist
URL: http://bugs.freeciv.org/Ticket/Display.html?id=18234 On 7/2/06, Billy Naylor [EMAIL PROTECTED] wrote: The help dialogues don't show if units require a specific building, the building help dialog shows what units it allows tho'. Patch - ML diff -Nurd -X.diff_ignore

[Freeciv-Dev] (PR#33626) City always sees submarines

2007-01-14 Thread Marko Lindqvist
URL: http://bugs.freeciv.org/Ticket/Display.html?id=33626 City sees submarine even when it's one square away. Battleship sees it only when in adjacent square. - ML ___ Freeciv-dev mailing list Freeciv-dev@gna.org

[Freeciv-Dev] (PR#33628) Can't make freeciv.pot for trunk

2007-01-14 Thread Marko Lindqvist
URL: http://bugs.freeciv.org/Ticket/Display.html?id=33628 TRUNK/build/po$ make freeciv.pot /usr/bin/xgettext --default-domain=freeciv --directory=../../src.patched \ --add-comments=TRANS: --escape --keyword=_ --keyword=N_ --keyword=Q_ --keyword=PL_:1,2 \ --language=C

Re: [Freeciv-Dev] (PR#18773) Reordering worklist smeshes it up if there is just bought unit

2007-01-15 Thread Marko Lindqvist
URL: http://bugs.freeciv.org/Ticket/Display.html?id=18773 On 7/21/06, Tommi Björkbacka [EMAIL PROTECTED] wrote: You can reproduce this problem easily. First buy a random unit and then put a new unit into worklist. After that try to adjust new unit to the top of worklist. Result is that you

Re: [Freeciv-Dev] (PR#18234) Feature request - Help dialog shows if units require a Building

2007-01-15 Thread Marko Lindqvist
URL: http://bugs.freeciv.org/Ticket/Display.html?id=18234 On 1/15/07, Marko Lindqvist [EMAIL PROTECTED] wrote: On 7/2/06, Billy Naylor [EMAIL PROTECTED] wrote: The help dialogues don't show if units require a specific building, the building help dialog shows what units it allows tho

[Freeciv-Dev] (PR#33761) [Bug] Editor allows placing units inside enemy city

2007-01-15 Thread Marko Lindqvist
URL: http://bugs.freeciv.org/Ticket/Display.html?id=33761 $subject - ML ___ Freeciv-dev mailing list Freeciv-dev@gna.org https://mail.gna.org/listinfo/freeciv-dev

[Freeciv-Dev] (PR#33762) [Patch] UCF_UNREACHABLE

2007-01-15 Thread Marko Lindqvist
URL: http://bugs.freeciv.org/Ticket/Display.html?id=33762 Only F_FIGHER can attack air units. It has been hardcoded to AIR_MOVING. This patch turns it into unit class flag. - ML diff -Nurd -X.diff_ignore freeciv/client/helpdata.c freeciv/client/helpdata.c --- freeciv/client/helpdata.c

[Freeciv-Dev] (PR#33767) gen-base

2007-01-16 Thread Marko Lindqvist
URL: http://bugs.freeciv.org/Ticket/Display.html?id=33767 By 'Base' I mean buildings outside cities: fortresses and airbases in current codebase. Some of their properties should come from ruleset instead of being hardcoded. Originally reason I started thinking this was that name 'Airbase' is

[Freeciv-Dev] (PR#33793) [Patch] Fix S2_0 gcc 4 compilation

2007-01-16 Thread Marko Lindqvist
URL: http://bugs.freeciv.org/Ticket/Display.html?id=33793 S2_0 does not compile without --disable-alsa with gcc 4. Or maybe problem can be in my alsa version? Anyway, attached patch fixes this. - ML diff -Nurd -X.diff_ignore freeciv/client/audio_alsa.c freeciv/client/audio_alsa.c ---

Re: [Freeciv-Dev] (PR#18234) Feature request - Help dialog shows if units require a Building

2007-01-16 Thread Marko Lindqvist
URL: http://bugs.freeciv.org/Ticket/Display.html?id=18234 On 1/16/07, Marko Lindqvist [EMAIL PROTECTED] wrote: On 1/15/07, Marko Lindqvist [EMAIL PROTECTED] wrote: On 7/2/06, Billy Naylor [EMAIL PROTECTED] wrote: The help dialogues don't show if units require a specific building

[Freeciv-Dev] (PR#33876) [Patch] tile_has_base_effect()

2007-01-17 Thread Marko Lindqvist
URL: http://bugs.freeciv.org/Ticket/Display.html?id=33876 This adds new module base.[ch]. It contains one function, base_has_effect(). New function tile_has_base_effect() in tile.c uses it. tile_has_base_effect() is called from number of places instead of directly checking if tile contains

Re: [Freeciv-Dev] (PR#18773) Reordering worklist smeshes it up if there is just bought unit

2007-01-17 Thread Marko Lindqvist
URL: http://bugs.freeciv.org/Ticket/Display.html?id=18773 On 1/15/07, Marko Lindqvist [EMAIL PROTECTED] wrote: On 7/21/06, Tommi Björkbacka [EMAIL PROTECTED] wrote: You can reproduce this problem easily. First buy a random unit and then put a new unit into worklist. After that try

Re: [Freeciv-Dev] Re: (PR#19822) bug in path_finding.c

2007-01-17 Thread Marko Lindqvist
URL: http://bugs.freeciv.org/Ticket/Display.html?id=19822 New approach. This patch removes segment_length and step completely. When segment or path length is needed, it's determined by extra iteration. I'm afraid this is unavoidable performance hit. This version of the patch is for S2_0

Re: [Freeciv-Dev] Re: (PR#19822) bug in path_finding.c

2007-01-17 Thread Marko Lindqvist
URL: http://bugs.freeciv.org/Ticket/Display.html?id=19822 On 1/17/07, Marko Lindqvist [EMAIL PROTECTED] wrote: This patch removes segment_length and step completely. When segment or path length is needed, it's determined by extra iteration. I'm afraid this is unavoidable performance hit

[Freeciv-Dev] (PR#33910) [Patch] Debug same nation bug

2007-01-17 Thread Marko Lindqvist
URL: http://bugs.freeciv.org/Ticket/Display.html?id=33910 Attached patch checks if two players have same nation and prints message if they do. I mean to commit this to both trunk and S2_1 to hunt down #17436. - ML diff -Nurd -X.diff_ignore freeciv/server/srv_main.c

Re: [Freeciv-Dev] (PR#33626) City always sees submarines

2007-01-17 Thread Marko Lindqvist
URL: http://bugs.freeciv.org/Ticket/Display.html?id=33626 On 1/15/07, Marko Lindqvist [EMAIL PROTECTED] wrote: City sees submarine even when it's one square away. Battleship sees it only when in adjacent square. Is this intentional? Anyway, attached patch changes that so that city sees

(PR#14490) Re: [Freeciv-Dev] (PR#33914) Bug Report

2007-01-17 Thread Marko Lindqvist
URL: http://bugs.freeciv.org/Ticket/Display.html?id=14490 On 1/18/07, [EMAIL PROTECTED] [EMAIL PROTECTED] wrote: 1: In file beta2.sav.gz, there is no entry in the registry for c7.ai.urgency (or the entries are out of order.) I think problem is that worklist length varies between cities.

(PR#14490) Re: [Freeciv-Dev] (PR#33914) Bug Report

2007-01-17 Thread Marko Lindqvist
URL: http://bugs.freeciv.org/Ticket/Display.html?id=14490 On 1/18/07, [EMAIL PROTECTED] [EMAIL PROTECTED] wrote: I can supply the saved game file, if you so desire That would be useful. Thanks. - ML ___ Freeciv-dev mailing list

Re: [Freeciv-Dev] (PR#32868) superbarbarians patch

2007-01-17 Thread Marko Lindqvist
URL: http://bugs.freeciv.org/Ticket/Display.html?id=32868 On 1/9/07, Daniel Markstedt [EMAIL PROTECTED] wrote: Updated the superbarbarians patch for dev (applies to S2_1 as well) + adding some helptext. The higher levels SERIOUSLY empowers the barbs: lvl 5 gives them twice the manpower;

Re: [Freeciv-Dev] (PR#33445) win32 client build broken (patch attached)

2007-01-18 Thread Marko Lindqvist
URL: http://bugs.freeciv.org/Ticket/Display.html?id=33445 From: Jonathan Landis [EMAIL PROTECTED] I wrote a patch for the Win32 client a couple of weeks ago. Thanks for the patch, Jonathan. I cannot compile and I have not even seen win32 client, but I looked some of the most obviously

[Freeciv-Dev] (PR#34035) [Patch] Fix memory leak in terrain.ruleset loading

2007-01-18 Thread Marko Lindqvist
URL: http://bugs.freeciv.org/Ticket/Display.html?id=34035 $subject - ML diff -Nurd -X.diff_ignore freeciv/server/ruleset.c freeciv/server/ruleset.c --- freeciv/server/ruleset.c 2006-08-18 10:51:27.0 +0300 +++ freeciv/server/ruleset.c 2007-01-19 04:23:36.0 +0200 @@ -1734,6

Re: [Freeciv-Dev] (PR#34032) wishlist: further cripple novice AI

2007-01-18 Thread Marko Lindqvist
URL: http://bugs.freeciv.org/Ticket/Display.html?id=34032 On 1/19/07, Daniel Markstedt [EMAIL PROTECTED] wrote: I suggest we cripple novice further so that it becomes what players expect it to be: something that even a novice can stand a chance against. It should build sparser empires,

Re: [Freeciv-Dev] (PR#34032) wishlist: further cripple novice AI

2007-01-19 Thread Marko Lindqvist
URL: http://bugs.freeciv.org/Ticket/Display.html?id=34032 On 1/19/07, Marko Lindqvist [EMAIL PROTECTED] wrote: About changing rules, attached is untested patch that halves firepower for novice AI. Not tested, but it breaks victory probability calcultations at client side. However, main

[Freeciv-Dev] (PR#34073) [Patch] H_DANGER

2007-01-19 Thread Marko Lindqvist
URL: http://bugs.freeciv.org/Ticket/Display.html?id=34073 This adds new AI handicap, H_DANGER. It causes AI to always cosider all it cities to be in grave danger. This causes it to spend some resources to unnecessary defense instead of expansion. Handicap is given to novice level AI. Maybe it

[Freeciv-Dev] (PR#34081) [Patch] Read base flags from ruleset

2007-01-19 Thread Marko Lindqvist
URL: http://bugs.freeciv.org/Ticket/Display.html?id=34081 Applies on top of #33876. - ML diff -Nurd -X.diff_ignore freeciv/common/base.c freeciv/common/base.c --- freeciv/common/base.c 2007-01-19 13:50:26.0 +0200 +++ freeciv/common/base.c 2007-01-19 13:57:54.0 +0200 @@

(PR#17436) Re: [Freeciv-Dev] (PR#33883) server select duplicate nation

2007-01-19 Thread Marko Lindqvist
URL: http://bugs.freeciv.org/Ticket/Display.html?id=17436 On 1/17/07, Daniel Markstedt [EMAIL PROTECTED] wrote: In recent trunk: Selected a fictional nation to play as. The server chose the very same nation for one of the AI players. Do you know exact server settings, including gameseed,

Re: [Freeciv-Dev] (PR#33761) [Bug] Editor allows placing units inside enemy city

2007-01-19 Thread Marko Lindqvist
URL: http://bugs.freeciv.org/Ticket/Display.html?id=33761 On 1/16/07, Marko Lindqvist [EMAIL PROTECTED] wrote: $subject Fix - ML diff -Nurd -X.diff_ignore freeciv/server/edithand.c freeciv/server/edithand.c --- freeciv/server/edithand.c 2006-07-17 23:56:55.0 +0300 +++ freeciv

[Freeciv-Dev] (PR#34095) [Patch] Nation sanity checking

2007-01-19 Thread Marko Lindqvist
URL: http://bugs.freeciv.org/Ticket/Display.html?id=34095 Some sanity checking for nations and players. For trunk only. - ML diff -Nurd -X.diff_ignore freeciv/common/player.c freeciv/common/player.c --- freeciv/common/player.c 2006-07-17 23:56:46.0 +0300 +++ freeciv/common/player.c

Re: [Freeciv-Dev] (PR#34095) [Patch] Nation sanity checking

2007-01-19 Thread Marko Lindqvist
URL: http://bugs.freeciv.org/Ticket/Display.html?id=34095 On 1/19/07, Marko Lindqvist [EMAIL PROTECTED] wrote: Some sanity checking for nations and players. For trunk only. One more assert added - ML diff -Nurd -X.diff_ignore freeciv/common/player.c freeciv/common/player.c --- freeciv

Re: (PR#17436) Re: [Freeciv-Dev] (PR#33883) server select duplicate nation

2007-01-19 Thread Marko Lindqvist
URL: http://bugs.freeciv.org/Ticket/Display.html?id=17436 handle_nation_select_req() always marks players previous nation available. This is not right when nation is not changed, but only leader name or sex. player_set_nation() sets nation-player correctly, so handle_nation_select_req() should

Re: (PR#17436) Re: [Freeciv-Dev] (PR#33883) server select duplicate nation

2007-01-19 Thread Marko Lindqvist
URL: http://bugs.freeciv.org/Ticket/Display.html?id=17436 On 1/19/07, Marko Lindqvist [EMAIL PROTECTED] wrote: handle_nation_select_req() always marks players previous nation available. This is not right when nation is not changed, but only leader name or sex. player_set_nation() sets

[Freeciv-Dev] (PR#34111) [Bug] Barbarian players share nation

2007-01-19 Thread Marko Lindqvist
URL: http://bugs.freeciv.org/Ticket/Display.html?id=34111 We have two barbarian players but only one nation which they share. This means that nation-player points to one of them, but for another pplayer-nation-player != pplayer. - ML ___

[Freeciv-Dev] (PR#34127) [Patch] Fix gui-stub compilation

2007-01-19 Thread Marko Lindqvist
URL: http://bugs.freeciv.org/Ticket/Display.html?id=34127 $subject - ML diff -Nurd -X.diff_ignore freeciv/client/gui-stub/mapview.c freeciv/client/gui-stub/mapview.c --- freeciv/client/gui-stub/mapview.c 2006-07-17 23:56:54.0 +0300 +++ freeciv/client/gui-stub/mapview.c 2007-01-20

[Freeciv-Dev] (PR#34131) [Patch] Split F_FIGHTER

2007-01-19 Thread Marko Lindqvist
URL: http://bugs.freeciv.org/Ticket/Display.html?id=34131 This splits Fighter flag in two. Fighter is very good attacking Helicopter units. AttackAny can attack even otherwise unreachable units. Applies on top of #33762 - ML diff -Nurd -X.diff_ignore freeciv/client/helpdata.c

[Freeciv-Dev] (PR#34133) [Patch] AI bomber vs bomber fix

2007-01-19 Thread Marko Lindqvist
URL: http://bugs.freeciv.org/Ticket/Display.html?id=34133 AI bomber might target enemy aircraft, which it cannot attack. Fix attached. - ML diff -Nurd -X.diff_ignore freeciv/ai/aiair.c freeciv/ai/aiair.c --- freeciv/ai/aiair.c 2006-08-18 10:52:03.0 +0300 +++ freeciv/ai/aiair.c

[Freeciv-Dev] (PR#34145) [Bug] Aiferry memory problem

2007-01-20 Thread Marko Lindqvist
URL: http://bugs.freeciv.org/Ticket/Display.html?id=34145 Autogames differ between executions. This is what valgrind found out: ==6086== Conditional jump or move depends on uninitialised value(s) ==6086==at 0x811ED99: ai_manage_ferryboat (aiferry.c:795) ==6086==by 0x812F8AD:

Re: [Freeciv-Dev] (PR#34111) [Bug] Barbarian players share nation

2007-01-20 Thread Marko Lindqvist
URL: http://bugs.freeciv.org/Ticket/Display.html?id=34111 On 1/19/07, Marko Lindqvist [EMAIL PROTECTED] wrote: We have two barbarian players but only one nation which they share. This means that nation-player points to one of them, but for another pplayer-nation-player != pplayer

[Freeciv-Dev] Re: (PR#34129) gui-sdl compile problem: round()

2007-01-20 Thread Marko Lindqvist
URL: http://bugs.freeciv.org/Ticket/Display.html?id=34129 On 1/20/07, Christian Prochaska [EMAIL PROTECTED] wrote: [EMAIL PROTECTED] - Sa 20. Jan 2007, 00:49:16]: cc1: warnings being treated as errors ../../../src.patched/client/gui-sdl/widget_scrollbar.c: In function

Re: [Freeciv-Dev] (PR#34111) [Bug] Barbarian players share nation

2007-01-20 Thread Marko Lindqvist
URL: http://bugs.freeciv.org/Ticket/Display.html?id=34111 On 1/20/07, Daniel Markstedt [EMAIL PROTECTED] wrote: On 1/20/07, Marko Lindqvist [EMAIL PROTECTED] wrote: On 1/19/07, Marko Lindqvist [EMAIL PROTECTED] wrote: We have two barbarian players but only one nation which they share

Re: [Freeciv-Dev] (PR#34111) [Bug] Barbarian players share nation

2007-01-20 Thread Marko Lindqvist
URL: http://bugs.freeciv.org/Ticket/Display.html?id=34111 On 1/20/07, Daniel Markstedt [EMAIL PROTECTED] wrote: Or we could use my Pirate nation. Agreed, the effect would be slightly humorous. But so is having Conestoga wagons in the year 4000 BC. ;) We cannot currently control which

Re: [Freeciv-Dev] (PR#34145) [Bug] Aiferry memory problem

2007-01-20 Thread Marko Lindqvist
URL: http://bugs.freeciv.org/Ticket/Display.html?id=34145 On 1/20/07, Marko Lindqvist [EMAIL PROTECTED] wrote: URL: http://bugs.freeciv.org/Ticket/Display.html?id=34145 Autogames differ between executions. This is what valgrind found out: ==6086== Conditional jump or move depends

Re: [Freeciv-Dev] (PR#33876) [Patch] tile_has_base_effect()

2007-01-20 Thread Marko Lindqvist
URL: http://bugs.freeciv.org/Ticket/Display.html?id=33876 On 1/19/07, Marko Lindqvist [EMAIL PROTECTED] wrote: On 1/17/07, Marko Lindqvist [EMAIL PROTECTED] wrote: This adds new module base.[ch]. It contains one function, base_has_effect(). New function tile_has_base_effect() in tile.c

Re: [Freeciv-Dev] (PR#34081) [Patch] Read base flags from ruleset

2007-01-20 Thread Marko Lindqvist
URL: http://bugs.freeciv.org/Ticket/Display.html?id=34081 On 1/19/07, Marko Lindqvist [EMAIL PROTECTED] wrote: Applies on top of #33876. - Improvet ruleset comments - Send information to client - ML diff -Nurd -X.diff_ignore freeciv/client/packhand.c freeciv/client/packhand.c

[Freeciv-Dev] (PR#34206) [Patch] Use nation name in uprising message

2007-01-20 Thread Marko Lindqvist
URL: http://bugs.freeciv.org/Ticket/Display.html?id=34206 Use barbarian nation name instead of hardcoded Barbarians in uprising message. - ML ___ Freeciv-dev mailing list Freeciv-dev@gna.org https://mail.gna.org/listinfo/freeciv-dev

Re: [Freeciv-Dev] (PR#34206) [Patch] Use nation name in uprising message

2007-01-20 Thread Marko Lindqvist
URL: http://bugs.freeciv.org/Ticket/Display.html?id=34206 On 1/21/07, Marko Lindqvist [EMAIL PROTECTED] wrote: Use barbarian nation name instead of hardcoded Barbarians in uprising message. And the patch... - ML diff -Nurd -X.diff_ignore freeciv/server/barbarian.c freeciv/server

[Freeciv-Dev] (PR#34224) Memory loss in ai diplomacy

2007-01-21 Thread Marko Lindqvist
URL: http://bugs.freeciv.org/Ticket/Display.html?id=34224 Form valgrind: ==3480== 1,036 (396 direct, 640 indirect) bytes in 35 blocks are definitely lost in loss record 2 of 6 ==3480==at 0x401C422: malloc (vg_replace_malloc.c:149) ==3480==by 0x804BD0D: fc_real_malloc (mem.c:82)

Re: [Freeciv-Dev] (PR#33445) win32 client build broken (patch attached)

2007-01-21 Thread Marko Lindqvist
URL: http://bugs.freeciv.org/Ticket/Display.html?id=33445 On 1/21/07, [EMAIL PROTECTED] [EMAIL PROTECTED] wrote: Except for cs.po, the win32 client build was ok. Thanks for testing. I have now committed move_type and worklist patches. There were a few warnings from gcc about things that

Re: [Freeciv-Dev] (PR#33876) [Patch] tile_has_base_flag()

2007-01-21 Thread Marko Lindqvist
URL: http://bugs.freeciv.org/Ticket/Display.html?id=33876 On 1/20/07, Marko Lindqvist [EMAIL PROTECTED] wrote: On 1/19/07, Marko Lindqvist [EMAIL PROTECTED] wrote: On 1/17/07, Marko Lindqvist [EMAIL PROTECTED] wrote: This adds new module base.[ch]. It contains one function

Re: [Freeciv-Dev] (PR#18466) Gfx problem with two sea types

2007-01-21 Thread Marko Lindqvist
URL: http://bugs.freeciv.org/Ticket/Display.html?id=18466 On 1/21/07, Marko Lindqvist [EMAIL PROTECTED] wrote: URL: http://bugs.freeciv.org/Ticket/Display.html?id=18466 On 7/10/06, Marko Lindqvist [EMAIL PROTECTED] wrote: I don't think it's possible with current codebase to create

Re: [Freeciv-Dev] (PR#18466) Gfx problem with two sea types

2007-01-21 Thread Marko Lindqvist
URL: http://bugs.freeciv.org/Ticket/Display.html?id=18466 On 1/21/07, Marko Lindqvist [EMAIL PROTECTED] wrote: On 1/21/07, Marko Lindqvist [EMAIL PROTECTED] wrote: URL: http://bugs.freeciv.org/Ticket/Display.html?id=18466 On 7/10/06, Marko Lindqvist [EMAIL PROTECTED] wrote: I

[Freeciv-Dev] (PR#34262) [Patch] Rename match_type other - land

2007-01-21 Thread Marko Lindqvist
URL: http://bugs.freeciv.org/Ticket/Display.html?id=34262 Most tilesets have two layer0 match_type: ocean and other. This patch renames other as land. Even if some terrain type later needs special match_type, it has to come in addition to land, since drawing second ocean type correctly will

[Freeciv-Dev] (PR#34272) [Patch] Deep Ocean gfx fallback

2007-01-21 Thread Marko Lindqvist
URL: http://bugs.freeciv.org/Ticket/Display.html?id=34272 Most tilesets handle Deep Ocean from #34265 using ocean graphics as fallback. Amplio and Isotrident however use wrong colour blending between Ocean and Deep Ocean tiles in that case. Attached patch defines Deep Ocean gfx separately for

[Freeciv-Dev] Re: (PR#34129) gui-sdl compile problem: round()

2007-01-21 Thread Marko Lindqvist
URL: http://bugs.freeciv.org/Ticket/Display.html?id=34129 On 1/21/07, Christian Prochaska [EMAIL PROTECTED] wrote: [EMAIL PROTECTED] - So 21. Jan 2007, 11:11:03]: On 1/21/07, Christian Prochaska [EMAIL PROTECTED] wrote: [EMAIL PROTECTED] - Sa 20. Jan 2007, 13:33:05]:

Re: [Freeciv-Dev] (PR#34276) clients default to the wrong port

2007-01-21 Thread Marko Lindqvist
URL: http://bugs.freeciv.org/Ticket/Display.html?id=34276 On 1/22/07, Daniel Markstedt [EMAIL PROTECTED] wrote: The default port has changed from to 5556, but the clients (tested with GTK2 and SDL) still default to . You probably have old port number stored in ~/.civclientrc -

Re: [Freeciv-Dev] (PR#32017) barbarian-only units

2007-01-22 Thread Marko Lindqvist
URL: http://bugs.freeciv.org/Ticket/Display.html?id=32017 On 1/7/07, Daniel Markstedt [EMAIL PROTECTED] wrote: This patch adds a Horse Archer as well as Elephants as barbarian-only units. The Horse Archer is available from the beginning, while the Elephant now requires Polytheism and

Re: [Freeciv-Dev] (PR#18466) Gfx problem with two sea types

2007-01-22 Thread Marko Lindqvist
URL: http://bugs.freeciv.org/Ticket/Display.html?id=18466 On 1/22/07, Daniel Markstedt [EMAIL PROTECTED] wrote: On 1/22/07, Marko Lindqvist [EMAIL PROTECTED] wrote: So this ticket is down to blending problem with amplio seas. 1) We want to blend between Ocean and land. 2) We want

Re: [Freeciv-Dev] (PR#18466) Gfx problem with two sea types

2007-01-22 Thread Marko Lindqvist
URL: http://bugs.freeciv.org/Ticket/Display.html?id=18466 On 1/22/07, Marko Lindqvist [EMAIL PROTECTED] wrote: On 1/22/07, Daniel Markstedt [EMAIL PROTECTED] wrote: On 1/22/07, Marko Lindqvist [EMAIL PROTECTED] wrote: So this ticket is down to blending problem with amplio seas. 1

Re: (PR#17436) [Freeciv-Dev] (PR#33883) server select duplicate nation

2007-01-22 Thread Marko Lindqvist
URL: http://bugs.freeciv.org/Ticket/Display.html?id=17436 On 1/22/07, Egor Vyscrebentsov [EMAIL PROTECTED] wrote: On 1/19/07, Marko Lindqvist wrote: handle_nation_select_req() always marks players previous nation available. This is not right when nation is not changed, but only

Re: [Freeciv-Dev] (PR#34111) [Bug] Barbarian players share nation

2007-01-22 Thread Marko Lindqvist
URL: http://bugs.freeciv.org/Ticket/Display.html?id=34111 On 1/20/07, Marko Lindqvist [EMAIL PROTECTED] wrote: On 1/20/07, Daniel Markstedt [EMAIL PROTECTED] wrote: Or we could use my Pirate nation. Agreed, the effect would be slightly humorous. But so is having Conestoga wagons

Re: [Freeciv-Dev] (PR#32017) barbarian-only units

2007-01-22 Thread Marko Lindqvist
URL: http://bugs.freeciv.org/Ticket/Display.html?id=32017 On 1/22/07, Daniel Markstedt [EMAIL PROTECTED] wrote: On 1/22/07, Marko Lindqvist [EMAIL PROTECTED] wrote: Can barbarians really build units with NoBuild flag? Only unit with NoBuild flag so far has been barbarian leader which

[Freeciv-Dev] (PR#34321) [Patch] Check that barbarian can build what it selects

2007-01-22 Thread Marko Lindqvist
URL: http://bugs.freeciv.org/Ticket/Display.html?id=34321 Barbarian building code just starts building units without checking if building that unit is at all possible in that city. So barbarian can for example end building ship in landlocked city, if someone gives ship a BARBARIAN_BUILD role.

[Freeciv-Dev] (PR#34379) [Patch] pft_fill_unit_default_parameters() and UCF_TERRAIN_SPEED

2007-01-23 Thread Marko Lindqvist
URL: http://bugs.freeciv.org/Ticket/Display.html?id=34379 This fixes pft_fill_unit_default_parameters() in respect of UCF_TERRAIN_SPEED. - ML diff -Nurd -X.diff_ignore freeciv/common/aicore/pf_tools.c freeciv/common/aicore/pf_tools.c --- freeciv/common/aicore/pf_tools.c 2007-01-22

Re: (PR#17436) Re: [Freeciv-Dev] (PR#33883) server select duplicate nation

2007-01-24 Thread Marko Lindqvist
URL: http://bugs.freeciv.org/Ticket/Display.html?id=17436 On 1/19/07, Marko Lindqvist [EMAIL PROTECTED] wrote: On 1/19/07, Marko Lindqvist [EMAIL PROTECTED] wrote: handle_nation_select_req() always marks players previous nation available. This is not right when nation is not changed

[Freeciv-Dev] (PR#34446) [Patch] sdl_get_tile_defense_info_text() BF_DEFENSE_BONUS

2007-01-24 Thread Marko Lindqvist
URL: http://bugs.freeciv.org/Ticket/Display.html?id=34446 Check against BF_DEFENSE_BONUS instead of presence of S_FORTRESS. - ML diff -Nurd -X.diff_ignore freeciv/client/gui-sdl/dialogs.c freeciv/client/gui-sdl/dialogs.c --- freeciv/client/gui-sdl/dialogs.c 2007-01-22 20:12:13.0

[Freeciv-Dev] (PR#34457) [Patch] UCF_CAN_OCCUPY units as Ransom collectors

2007-01-24 Thread Marko Lindqvist
URL: http://bugs.freeciv.org/Ticket/Display.html?id=34457 Instead of checking if is_ground_unit() or is_heli_unit(), check for UCF_CAN_OCCUPY. Using same flag for two different things might be bad, but creating separate flag just for this seemed waste. - ML diff -Nurd -X.diff_ignore

Re: [Freeciv-Dev] (PR#34463) [Bug] Sea barbarians inactive

2007-01-25 Thread Marko Lindqvist
URL: http://bugs.freeciv.org/Ticket/Display.html?id=34463 On 1/25/07, Marko Lindqvist [EMAIL PROTECTED] wrote: On 1/24/07, Marko Lindqvist [EMAIL PROTECTED] wrote: On 1/24/07, Marko Lindqvist [EMAIL PROTECTED] wrote: Seen as global observer: Barbarian boats almost never move

Re: (PR#17436) Re: [Freeciv-Dev] (PR#33883) server select duplicate nation

2007-01-25 Thread Marko Lindqvist
URL: http://bugs.freeciv.org/Ticket/Display.html?id=17436 On 1/24/07, Marko Lindqvist [EMAIL PROTECTED] wrote: On 1/19/07, Marko Lindqvist [EMAIL PROTECTED] wrote: On 1/19/07, Marko Lindqvist [EMAIL PROTECTED] wrote: handle_nation_select_req() always marks players previous nation

[Freeciv-Dev] (PR#34551) [Patch] Define ferry.ai.passenger value 0 as FERRY_ABANDON_BOSS

2007-01-25 Thread Marko Lindqvist
URL: http://bugs.freeciv.org/Ticket/Display.html?id=34551 It took me a while to understand why ferry.ai.passenger sometimes get value 0 instead of FERRY_AVAILABLE. Attached patch makes that a bit clearer. - ML diff -Nurd -X.diff_ignore freeciv/ai/aiferry.c freeciv/ai/aiferry.c ---

Re: [Freeciv-Dev] (PR#34321) [Patch] Check that barbarian can build what it selects

2007-01-25 Thread Marko Lindqvist
URL: http://bugs.freeciv.org/Ticket/Display.html?id=34321 On 1/22/07, Marko Lindqvist [EMAIL PROTECTED] wrote: Barbarian building code just starts building units without checking if building that unit is at all possible in that city. So barbarian can for example end building ship

Re: [Freeciv-Dev] (PR#34463) [Bug] Sea barbarians inactive

2007-01-25 Thread Marko Lindqvist
URL: http://bugs.freeciv.org/Ticket/Display.html?id=34463 On 1/25/07, Marko Lindqvist [EMAIL PROTECTED] wrote: On 1/25/07, Marko Lindqvist [EMAIL PROTECTED] wrote: On 1/24/07, Marko Lindqvist [EMAIL PROTECTED] wrote: On 1/24/07, Marko Lindqvist [EMAIL PROTECTED] wrote: Seen

[Freeciv-Dev] (PR#34563) [Patch] Fix ai_activate_passengers() crash

2007-01-25 Thread Marko Lindqvist
URL: http://bugs.freeciv.org/Ticket/Display.html?id=34563 ai_activate_passengers() need to use unit_list_iterate_safe() as managed unit might die or move out of tile. - ML diff -Nurd -X.diff_ignore freeciv/ai/aiferry.c freeciv/ai/aiferry.c --- freeciv/ai/aiferry.c 2006-07-17

Re: [Freeciv-Dev] (PR#34463) [Bug] Sea barbarians inactive

2007-01-25 Thread Marko Lindqvist
URL: http://bugs.freeciv.org/Ticket/Display.html?id=34463 On 1/25/07, Marko Lindqvist [EMAIL PROTECTED] wrote: On 1/25/07, Marko Lindqvist [EMAIL PROTECTED] wrote: On 1/25/07, Marko Lindqvist [EMAIL PROTECTED] wrote: On 1/24/07, Marko Lindqvist [EMAIL PROTECTED] wrote

Re: [Freeciv-Dev] (PR#34379) [Patch] pft_fill_unit_default_parameters() and UCF_TERRAIN_SPEED

2007-01-25 Thread Marko Lindqvist
URL: http://bugs.freeciv.org/Ticket/Display.html?id=34379 On 1/23/07, Marko Lindqvist [EMAIL PROTECTED] wrote: This fixes pft_fill_unit_default_parameters() in respect of UCF_TERRAIN_SPEED. Improvet comments a bit. - ML diff -Nurd -X.diff_ignore freeciv/common/aicore/pf_tools.c

[Freeciv-Dev] (PR#34566) [Patch] AI level name cleanup

2007-01-25 Thread Marko Lindqvist
URL: http://bugs.freeciv.org/Ticket/Display.html?id=34566 Taken and further developet from #34431. Introduced enum ai_level. Added functions find_ai_level_by_name(), ai_level_name() and ai_level_name_cmd() to common/player.c. Used where applicable. - ML diff -Nurd -X.diff_ignore

[Freeciv-Dev] (PR#34568) [Patch] Unit flag BarbarianOnly

2007-01-25 Thread Marko Lindqvist
URL: http://bugs.freeciv.org/Ticket/Display.html?id=34568 New unit type flag BarbarianOnly. These units can be built only by barbarians. - ML diff -Nurd -X.diff_ignore freeciv/client/helpdata.c freeciv/client/helpdata.c --- freeciv/client/helpdata.c 2007-01-23 16:05:15.0 +0200 +++

Re: [Freeciv-Dev] (PR#34336) [Patch] Base cleanup

2007-01-26 Thread Marko Lindqvist
URL: http://bugs.freeciv.org/Ticket/Display.html?id=34336 On 1/22/07, Marko Lindqvist [EMAIL PROTECTED] wrote: This introduces several new base related helper functions hiding the fact that bases are actually bits in specials bitvector. Used them where appropriate. Fixed a bug

[Freeciv-Dev] (PR#34647) [Patch] UCF_ZOC

2007-01-26 Thread Marko Lindqvist
URL: http://bugs.freeciv.org/Ticket/Display.html?id=34647 New unit class flag ZOC to mark classes that are subject to ZOC rules. Previously this was hardcoded to LAND_MOVING. Unit type flag IgZOC may override this. Now amphibious units too can be subject to ZOC rules. - ML diff -Nurd

[Freeciv-Dev] (PR#34713) [Patch] S2_1 Barbarian player name from correct nation

2007-01-28 Thread Marko Lindqvist
URL: http://bugs.freeciv.org/Ticket/Display.html?id=34713 If there is more than one barbarian nation, player name is chosen from wrong nation when barbarians reborn. Fix for S2_1 attached. #34111 will fix this for trunk. - ML diff -Nurd -X.diff_ignore freeciv/server/barbarian.c

Re: [Freeciv-Dev] (PR#34566) [Patch] AI level name cleanup

2007-02-01 Thread Marko Lindqvist
URL: http://bugs.freeciv.org/Ticket/Display.html?id=34566 On 1/27/07, Marko Lindqvist [EMAIL PROTECTED] wrote: On 1/27/07, Marko Lindqvist [EMAIL PROTECTED] wrote: On 1/26/07, Marko Lindqvist [EMAIL PROTECTED] wrote: Introduced enum ai_level. Added functions find_ai_level_by_name

Re: [Freeciv-Dev] (PR#34725) [Patch] AI accepts slightly negative balance if it has a lot of gold

2007-02-02 Thread Marko Lindqvist
URL: http://bugs.freeciv.org/Ticket/Display.html?id=34725 On 2/1/07, Marko Lindqvist [EMAIL PROTECTED] wrote: On 1/28/07, Marko Lindqvist [EMAIL PROTECTED] wrote: AI accepts slightly negative balance if it has gold reserves for that. More precisely: negative balance is acceptable if AI

[Freeciv-Dev] (PR#35708) [Patch] Triremes cannot enter Deep Ocean

2007-02-11 Thread Marko Lindqvist
URL: http://bugs.freeciv.org/Ticket/Display.html?id=35708 Untested patch to limit Triremes to Ocean tiles. Applies on top of Deep Ocean patch #34265. Also, Triremes do not get move bonus from Nuclear Power. - ML diff -Nurd -X.diff_ignore freeciv/data/default/effects.ruleset

Re: (PR#14490) Re: [Freeciv-Dev] (PR#33914) Bug Report

2007-02-11 Thread Marko Lindqvist
URL: http://bugs.freeciv.org/Ticket/Display.html?id=14490 On 1/18/07, Marko Lindqvist [EMAIL PROTECTED] wrote: On 1/18/07, [EMAIL PROTECTED] [EMAIL PROTECTED] wrote: 1: In file beta2.sav.gz, there is no entry in the registry for c7.ai.urgency (or the entries are out of order.) I

Re: (PR#14490) Re: [Freeciv-Dev] (PR#33914) Bug Report

2007-02-12 Thread Marko Lindqvist
URL: http://bugs.freeciv.org/Ticket/Display.html?id=14490 On 2/11/07, Marko Lindqvist [EMAIL PROTECTED] wrote: On 1/18/07, Marko Lindqvist [EMAIL PROTECTED] wrote: I think problem is that worklist length varies between cities. Untested patch attached. This version saves as many

Re: [Freeciv-Dev] (PR#18430) Pirate nation

2007-02-28 Thread Marko Lindqvist
URL: http://bugs.freeciv.org/Ticket/Display.html?id=18430 What is holding this ticket from being applied? Several other tickets depend on it (directly or indirectly). - ML ___ Freeciv-dev mailing list Freeciv-dev@gna.org

Re: [Freeciv-Dev] (PR#33499) [Bug] Editor creating broken cities for other players

2007-03-01 Thread Marko Lindqvist
URL: http://bugs.freeciv.org/Ticket/Display.html?id=33499 On 1/14/07, Marko Lindqvist [EMAIL PROTECTED] wrote: 1. Insert city owned by somebody else in editor mode. 2. Start another client as owner of the created city. 3. Try to look at the city. civclient: ../../src.patched/client

Re: [Freeciv-Dev] (PR#34695) [Patch] Cheating AI level

2007-03-02 Thread Marko Lindqvist
URL: http://bugs.freeciv.org/Ticket/Display.html?id=34695 On 3/1/07, Marko Lindqvist [EMAIL PROTECTED] wrote: URL: http://bugs.freeciv.org/Ticket/Display.html?id=34695 This adds new AI level 'cheating'. It is codewise just like 'hard', but distributed rulesets provide some cheating

  1   2   3   4   5   6   7   8   9   10   >