🚀 go-pugleaf

Roger Uzun (uzun@crash.cts.com) wrote:
: When KILLT is defined, new game should clear it out and do
: an Initialize_dist near the end.  That is NewGame ().
: Also GetGame should probably clear out killt[] and call initialize_dist,
: or perhaps just clear out killt[].

: -Roger
: --------------------------------------------------------------
: bix: ruzun
: NET: uzun@crash.cts.com

This is not actually necessary as killt[] is always kept in its original
state.  Killt is only changed in genmoves.c and these changes are always
undone before exiting.

However Roger also pointed out some init problems.  He suggested that
QueenCheck[0] should be set to 0.  We should probably do the same to
ChkFlag[0] too.  Another more serious problem is the PawnStorm static
variable.  This is set to false once and never again.  This would mean
that subsequent games could start with a PawnStorm of true which is
wrong.  The simplest way is to remove the static declaration and always
assume PawnStorm is false.  The calculation for PawnStorm status is
very very cheap after all.

Kong Sian