http://bugs.freeciv.org/Ticket/Display.html?id=39563 >
> [EMAIL PROTECTED] - Tue Feb 12 22:42:03 2008]:
>
>
/**
> -...
>
-**/
> -void copy_if_better_ch
http://bugs.freeciv.org/Ticket/Display.html?id=39563 >
/**
-...
-**/
-void copy_if_better_choice(struct ai_choice *cur, struct ai_choice *best)
-{
- if
http://bugs.freeciv.org/Ticket/Display.html?id=39563 >
OK, seems reasonable. I'd say commit.
___
Freeciv-dev mailing list
Freeciv-dev@gna.org
https://mail.gna.org/listinfo/freeciv-dev
http://bugs.freeciv.org/Ticket/Display.html?id=39563 >
On 10/02/2008, William Allen Simpson wrote:
>
> Here's my proposed patch to rid us of this troublesome beast
I'd rather keep the function. Even though 'better' choice is
currently determined simply by comparing want value, that's not
n
http://bugs.freeciv.org/Ticket/Display.html?id=39563 >
Here's my proposed patch to rid us of this troublesome beast
Index: ai/advdomestic.c
===
--- ai/advdomestic.c(revision 14391)
+++ ai/advdomestic.c(working copy)
@@ -
http://bugs.freeciv.org/Ticket/Display.html?id=39563 >
Thank you for finding my mistake in PR#39553!
This worthless function is only called 4 places, and the first place I've
looked is:
ai/advmilitary.c:1223:
if (best_choice.want > choice->want) {
/* We want attacker more than what we
http://bugs.freeciv.org/Ticket/Display.html?id=39563 >
Fix to actual bug. Needs some more testing to check if this is the
only problem.
- ML
diff -Nurd -X.diff_ignore freeciv/ai/aitools.c freeciv/ai/aitools.c
--- freeciv/ai/aitools.c 2008-01-20 04:02:14.0 +0200
+++ freeciv/ai/aitools
http://bugs.freeciv.org/Ticket/Display.html?id=39563 >
On Feb 4, 2008 2:28 PM, William Allen Simpson
<[EMAIL PROTECTED]> wrote:
> As Jason notes, the problem is that Per set the log level to LOG_ERROR,
> but the comments say this is OK
Indeed. In the absence of anyone to fix it, changing i
http://bugs.freeciv.org/Ticket/Display.html?id=39563 >
Found the old report and merged with it. Added some PR# links, instead
of the revision numbers Marko used in the report.
Funny, I've been waiting for Marko to fix this, as it's his original
report, but now I see that Per flagged it for me
http://bugs.freeciv.org/Ticket/Display.html?id=39563 >
On 14/08/07, Marko Lindqvist <[EMAIL PROTECTED]> wrote:
>
> Autogames produce only tons of:
> 1: Maria Theresia's Graz(54,37) [s2 d0 u0 g0] Falling back - didn't
> want to build soldiers, workers, caravans, settlers, or buildings!
>
> with
http://bugs.freeciv.org/Ticket/Display.html?id=39563 >
On 14/08/07, Marko Lindqvist <[EMAIL PROTECTED]> wrote:
>
> Autogames produce only tons of:
> 1: Maria Theresia's Graz(54,37) [s2 d0 u0 g0] Falling back - didn't
> want to build soldiers, workers, caravans, settlers, or buildings!
>
> with
http://bugs.freeciv.org/Ticket/Display.html?id=39563 >
Autogames produce only tons of:
1: Maria Theresia's Graz(54,37) [s2 d0 u0 g0] Falling back - didn't
want to build soldiers, workers, caravans, settlers, or buildings!
with actual player/city varying, of course.
- ML
_
12 matches
Mail list logo