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

["Per I. Mathisen via RT" <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
needed.

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

  - Per