Complete.Org: Mailing Lists: Archives: freeciv-ai: December 2002:
[freeciv-ai] Re: [Freeciv-Dev] (PR#2634) ai_military_attack() bug#2

[freeciv-ai] Re: [Freeciv-Dev] (PR#2634) ai_military_attack() bug#2

[Top] [All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index] [Thread Index]
Cc: freeciv-ai@xxxxxxxxxxx
Subject: [freeciv-ai] Re: [Freeciv-Dev] (PR#2634) ai_military_attack() bug#2
From: "Per I. Mathisen via RT" <rt@xxxxxxxxxxxxxx>
Date: Mon, 23 Dec 2002 10:46:27 -0800
Reply-to: rt@xxxxxxxxxxxxxx

On Mon, 23 Dec 2002, Gregory Berkolaiko via RT wrote:
> > It is possible to attack and die, and still do another loop in
> > ai_military_attack, leading to a core dump. If we add the first check, we
> This looks quite critical to me. Why haven't I seen any coredumps yet?

Because bug #1 ensured this code was almost never called.

> > can remove the latter. This should be fixed for S1_14 as well as head, but
> > for S1_14 we can keep the latter check as well to be absolutely sure.
> So beta4 ? ;)

No, by keeping both checks we do it failsafe. So a new beta won't be

But we might not want to fix bug#1 for S1_14... I dunno.

  - Per

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