Complete.Org: Mailing Lists: Archives: freeciv-dev: October 2005:
[Freeciv-Dev] Re: (PR#14457) S2_0: researched tech reported incorrectly
Home

[Freeciv-Dev] Re: (PR#14457) S2_0: researched tech reported incorrectly

[Top] [All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index] [Thread Index]
To: yautja@xxxxxxxxxxxxxxx
Subject: [Freeciv-Dev] Re: (PR#14457) S2_0: researched tech reported incorrectly
From: "Jason Short" <jdorje@xxxxxxxxxxxxxxxxxxxxx>
Date: Wed, 26 Oct 2005 01:24:48 -0700
Reply-to: bugs@xxxxxxxxxxx

<URL: http://bugs.freeciv.org/Ticket/Display.html?id=14457 >

Christian Knoke wrote:
> <URL: http://bugs.freeciv.org/Ticket/Display.html?id=14457 >
> 
> On Tue, Oct 25, 2005 at 03:38:37PM -0700, Brendon wrote:
> 
>><URL: http://bugs.freeciv.org/Ticket/Display.html?id=14457 >
>>
>>SVN S2_0 26 Oct GTK2 Linux
>>
>>Doesn't report the researched tech, but reports the current target twice 
>>instead. Not present in 2.06.

It was broken in this commit:

r11129 | mstefek | 2005-10-14 05:41:15 -0400 (Fri, 14 Oct 2005) | 1 line

When target is A_UNSET and goal is specified pick correct tech at the 
start of next turn. Reported by Jason, patch by me in PR#14288

this patch should fix it (untested).

-jason

Index: server/plrhand.c
===================================================================
--- server/plrhand.c    (revision 11191)
+++ server/plrhand.c    (working copy)
@@ -379,7 +379,7 @@
       notify_player_ex(plr, NULL, E_TECH_LEARNED,
                       _("Game: Learned %s.  "
                         "Our scientists focus on %s, goal is %s."),
-                      get_tech_name(plr, next_tech),
+                      get_tech_name(plr, tech_found),
                       get_tech_name(plr, plr->research.researching),
                       get_tech_name(plr, plr->ai.tech_goal));
     } else {

[Prev in Thread] Current Thread [Next in Thread]