Fix disambiguation for one-click moving
authorH.G.Muller <hgm@hgm-xboard.(none)>
Tue, 29 Mar 2016 18:20:54 +0000 (20:20 +0200)
committerH.G.Muller <hgm@hgm-xboard.(none)>
Tue, 29 Mar 2016 18:27:14 +0000 (20:27 +0200)
commit2d18618a1697830a12fbdcaa398bb8bde18558fe
tree5b406ae11b12815f85ce1705f968980d48d37287
parent9c85ff7d902909d866b59ac3d390aea310c01396
Fix disambiguation for one-click moving

The test in the DisambiguateCallback to ignore duplicate moves (as
sometimes generated by Betza descriptions) was only testing for the
same from square. This is good enugh if the to-square is always given
(as in SAN), so that moves with different to-square would always be
rejected anyway. But to determine if a piece has only a single move
we disambiguate with unknown to-square. So all moves but the first
of a piece would be ignored, making it always pass the only-move test.
We now test both from- and to-square, and only ignore moves that
have both of these equal to that of an already matching move.
moves.c