Re: [XBoard-devel] master-branch needs some testing

[h.g. muller]

At 20:41 7-10-2010 -0700, Arun Persaud wrote:
> Hi
>
> there are a several new commits that changed code all over the place
> (mostly to prevent buffer overflows). Would be great, if people could
> test the current master ;)
>
> Arun

Apart from this (which potentially could cause problems anywere) there were
two specific elaborate code changes, which deserve special testing:

The algorithm for handling the -mps, -tc and -inc parameters was completely 
changed,
in preparation for handling multi-session Time Controls in engine play.
So it would be good to check if every TC type that used to work still works,
also when the -tc parameter is of the form "min:sec", and if the flag is called
at the proper moment in all types of TC. I guess t should be pretty obvious
if something is wrong here, as there are not too many combinations.

The second change is in the the promotion code. The old code did not
protect well against erroneous or spurious promotion characters printed by 
engines
from leaking to the oher engine, or into the PGN, and vice versa. Also the 
default
of promoting to Queen when the promotion suffix was missing was no longer 
correct.
To cure all that the internal representation of promotions was changed.
Of special concern in variants with funny promotion rules (e.g. Shatranj,
Shogi, mini-Shogi). Also untested is if promotion with normal Chess in ICS 
play
still works.