Clean up fishy patch
authorH.G. Muller <h.g.muller@hccnet.nl>
Sat, 29 Jan 2011 18:59:11 +0000 (19:59 +0100)
committerArun Persaud <arun@nubati.net>
Sat, 29 Jan 2011 21:37:14 +0000 (13:37 -0800)
commit37171a1530eef8fbc24f3b656397d91164c43d61
treecb83d85dbbd2daffa08bd188ee5e068164c80dbc
parent50834a5d1deb1fa3f3edd73c61be0e55b89a637d
Clean up fishy patch

None of the ShowMove() calls in Adjudicate() was working, because when
this code was taken out of HandleMachineMove() it was taken outside the
scope of the local fromX, fromY etc. there, so that it started using the
global variables from the mouse driver. As these usually contain -1,
this suppressed move animation and highlighting. All these calls have
now been removed, and ShowMove() is called after a positive return (from
HandleMachineMove(), so within the correct scope).
This improves the previous fix, which led to some flicker, because the
erroneous ShowMove() calls did draw the new board (instantly), wich in
the fishy patch was then immediately overwritten by the old position, to
start animation. This drawing of the old position is now no longer
needed either.
backend.c