Re: [gnugo-devel] defend both, attack either, worms[], dragons[]

[Arend Bayer]

> retrieving revision 1.104
> diff -u -r1.104 move_reasons.c
> --- engine/move_reasons.c     11 Dec 2002 07:42:19 -0000      1.104
> +++ engine/move_reasons.c     11 Dec 2002 15:11:43 -0000
> @@ -646,9 +646,9 @@
>  void
>  add_attack_move(int pos, int ww, int code)
>  {
> +  ASSERT_ON_BOARD1(ww);
>    ww = worm[ww].origin;
>
> -  ASSERT_ON_BOARD1(ww);
>    if (code == WIN)
>      add_move_reason(pos, ATTACK_MOVE, ww);
>    else if (code == KO_A)

Hmm, I thought putting the ASSERT afterwards would also catch the cases
where (ww) points to an empty point on the board (as worm[].origin will
be zero there (and well, if it's way off board, there is at least a
very good chance that we will catch it...).

Arend