Reject more invalid moves.
authorSteinar H. Gunderson <sgunderson@bigfoot.com>
Tue, 15 Nov 2016 18:10:20 +0000 (19:10 +0100)
committerSteinar H. Gunderson <sgunderson@bigfoot.com>
Tue, 15 Nov 2016 18:10:20 +0000 (19:10 +0100)
Board.pm

index 3e0fd33..7201898 100644 (file)
--- a/Board.pm
+++ b/Board.pm
@@ -200,15 +200,14 @@ sub parse_pretty_move {
                        push @squares, [ $row, $col ];
                }
        }
-       if (scalar @squares > 1) {
-               # Filter out pieces which cannot reach this square.
-               @squares = grep { $board->can_reach($piece, $_->[0], $_->[1], $to_row, $to_col) } @squares;
-       }
-       if (scalar @squares > 1) {
-               # See if doing this move would put us in check
-               # (yes, there are clients that expect us to do this).
-               @squares = grep { !$board->make_move($_->[0], $_->[1], $to_row, $to_col, $promo)->in_check($side) } @squares;
-       }
+
+       # Filter out pieces which cannot reach this square.
+       @squares = grep { $board->can_reach($piece, $_->[0], $_->[1], $to_row, $to_col) } @squares;
+
+       # See if doing this move would put us in check
+       # (yes, there are clients that expect us to do this).
+       @squares = grep { !$board->make_move($_->[0], $_->[1], $to_row, $to_col, $promo)->in_check($side) } @squares;
+
        if (scalar @squares == 0) {
                die "Impossible move $move";
        }