We do not need to prettyprint a move if we already have the pretty form.

[remoteglot] / Position.pm

diff --git a/Position.pm b/Position.pm
index 9333bcd58af514badb9a7b31463a4c00763da609..272cadb78d2124275556e16df5bc7403b19cbe6d 100644 (file)
--- a/Position.pm
+++ b/Position.pm
@@ -28,6 +28,8 @@ sub new {
        $pos->{'player_b'} = $x[18];
        $pos->{'player_w'} =~ s/^W?[FCIG]M//;
        $pos->{'player_b'} =~ s/^W?[FCIG]M//;
        $pos->{'player_b'} = $x[18];
        $pos->{'player_w'} =~ s/^W?[FCIG]M//;
        $pos->{'player_b'} =~ s/^W?[FCIG]M//;

+       $pos->{'white_clock'} = $x[24];
+       $pos->{'black_clock'} = $x[25];

        $pos->{'move_num'} = $x[26];
        if ($x[27] =~ /([a-h][1-8])-([a-h][1-8])/) {
                $pos->{'last_move_uci'} = $1 . $2;
        $pos->{'move_num'} = $x[26];
        if ($x[27] =~ /([a-h][1-8])-([a-h][1-8])/) {
                $pos->{'last_move_uci'} = $1 . $2;


@@ -73,27 +75,12 @@ sub fen {
        my $ep = "-";
        if ($pos->{'ep_file_num'} != -1) {
                my $col = $pos->{'ep_file_num'};
        my $ep = "-";
        if ($pos->{'ep_file_num'} != -1) {
                my $col = $pos->{'ep_file_num'};

-               my $nep = (qw(a b c d e f g h))[$col];
+               $ep = (qw(a b c d e f g h))[$col];

 
                if ($pos->{'toplay'} eq 'B') {
 
                if ($pos->{'toplay'} eq 'B') {

-                       $nep .= "3";
+                       $ep .= "3";

                } else {
                } else {

-                       $nep .= "6";
-               }
-
-               #
-               # Showing the en passant square when actually no capture can be made
-               # seems to confuse at least Rybka. Thus, check if there's actually
-               # a pawn of the opposite side that can do the en passant move, and if
-               # not, just lie -- it doesn't matter anyway. I'm unsure what's the
-               # "right" thing as per the standard, though.
-               #
-               if ($pos->{'toplay'} eq 'B') {
-                       $ep = $nep if ($col > 0 && $pos->{'board'}[4][$col-1] eq 'p');
-                       $ep = $nep if ($col < 7 && $pos->{'board'}[4][$col+1] eq 'p');
-               } else {
-                       $ep = $nep if ($col > 0 && $pos->{'board'}[3][$col-1] eq 'P');
-                       $ep = $nep if ($col < 7 && $pos->{'board'}[3][$col+1] eq 'P');
+                       $ep .= "6";

                }
        }
        $fen .= " ";
                }
        }
        $fen .= " ";


@@ -110,6 +97,35 @@ sub fen {
        return $fen;
 }
 
        return $fen;
 }
 

+# Returns a compact bit string describing the same data as fen(),
+# except for the half-move and full-move clock.
+sub bitpacked_fen {
+       my $pos = shift;
+       my $board = $pos->{'board'}->bitpacked_fen();
+
+       my $bits = "";
+       if ($pos->{'toplay'} eq 'W') {
+               $bits .= "0";
+       } else {
+               $bits .= "1";
+       }
+
+       $bits .= $pos->{'white_castle_k'};
+       $bits .= $pos->{'white_castle_q'};
+       $bits .= $pos->{'black_castle_k'};
+       $bits .= $pos->{'black_castle_q'};
+
+       my $col = $pos->{'ep_file_num'};
+       if ($col == -1) {
+               $bits .= "0";
+       } else {
+               $bits .= "1";
+               $bits .= (qw(000 001 010 011 100 101 110 111))[$col];
+       }
+
+       return $board . pack('b*', $bits);
+}
+

 sub to_json_hash {
        my $pos = shift;
        my $json = { %$pos, fen => $pos->fen() };
 sub to_json_hash {
        my $pos = shift;
        my $json = { %$pos, fen => $pos->fen() };


@@ -141,7 +157,7 @@ sub num_pieces {
 
 # Returns a new Position object.
 sub make_move {
 
 # Returns a new Position object.
 sub make_move {

-        my ($pos, $from_row, $from_col, $to_row, $to_col, $promo) = @_;
+        my ($pos, $from_row, $from_col, $to_row, $to_col, $promo, $pretty_move) = @_;

 
        my $from_square = _pos_to_square($from_row, $from_col);
        my $to_square = _pos_to_square($to_row, $to_col);
 
        my $from_square = _pos_to_square($from_row, $from_col);
        my $to_square = _pos_to_square($to_row, $to_col);


@@ -195,8 +211,12 @@ sub make_move {
        }
        $np->{'player_w'} = $pos->{'player_w'};
        $np->{'player_b'} = $pos->{'player_b'};
        }
        $np->{'player_w'} = $pos->{'player_w'};
        $np->{'player_b'} = $pos->{'player_b'};

-       my ($move, $nb) = $pos->{'board'}->prettyprint_move($from_row, $from_col, $to_row, $to_col, $promo);
-       $np->{'last_move'} = $move;
+       if (defined($pretty_move)) {
+               $np->{'last_move'} = $pretty_move;
+       } else {
+               my ($move, $nb) = $pos->{'board'}->prettyprint_move($from_row, $from_col, $to_row, $to_col, $promo);
+               $np->{'last_move'} = $move;
+       }

        $np->{'last_move_uci'} = Board::move_to_uci_notation($from_row, $from_col, $to_row, $to_col, $promo);
 
        return bless $np;
        $np->{'last_move_uci'} = Board::move_to_uci_notation($from_row, $from_col, $to_row, $to_col, $promo);
 
        return bless $np;