X-Git-Url: https://git.sesse.net/?p=stockfish;a=blobdiff_plain;f=src%2Fmove.cpp;h=41d1603771a7b4d7d29daa019c23667deff99272;hp=e52a8e59e688853fad3557314e5850da63b62d71;hb=9c7e2c8f9d3eba45c20529ad0624e6de10fc7b83;hpb=2a21543c88f82a64ee48d00d706db102c0c4637d

diff --git a/src/move.cpp b/src/move.cpp
index e52a8e59..41d16037 100644
--- a/src/move.cpp
+++ b/src/move.cpp
@@ -26,9 +26,9 @@
 using std::string;
 
 /// move_to_uci() converts a move to a string in coordinate notation
-/// (g1f3, a7a8q, etc.). The only special case is castling moves, where we
-/// print in the e1g1 notation in normal chess mode, and in e1h1 notation in
-/// Chess960 mode. Instead internally Move is coded as "king captures rook".
+/// (g1f3, a7a8q, etc.). The only special case is castling moves, where we print
+/// in the e1g1 notation in normal chess mode, and in e1h1 notation in chess960
+/// mode. Internally castle moves are always coded as "king captures rook".
 
 const string move_to_uci(Move m, bool chess960) {
 
@@ -43,20 +43,22 @@ const string move_to_uci(Move m, bool chess960) {
       return "0000";
 
   if (is_castle(m) && !chess960)
-      to = from + (file_of(to) == FILE_H ? Square(2) : -Square(2));
+      to = make_square(to > from ? FILE_G : FILE_C, rank_of(from));
 
   if (is_promotion(m))
-      promotion = char(tolower(piece_type_to_char(promotion_piece_type(m))));
+      promotion = char(tolower(piece_type_to_char(promotion_type(m))));
 
   return square_to_string(from) + square_to_string(to) + promotion;
 }
 
 
 /// move_from_uci() takes a position and a string representing a move in
-/// simple coordinate notation and returns an equivalent Move if any.
-/// Moves are guaranteed to be legal.
+/// simple coordinate notation and returns an equivalent legal Move if any.
 
-Move move_from_uci(const Position& pos, const string& str) {
+Move move_from_uci(const Position& pos, string& str) {
+
+  if (str.length() == 5) // Junior could send promotion piece in uppercase
+      str[4] = char(tolower(str[4]));
 
   for (MoveList<MV_LEGAL> ml(pos); !ml.end(); ++ml)
       if (str == move_to_uci(ml.move(), pos.is_chess960()))
@@ -66,9 +68,8 @@ Move move_from_uci(const Position& pos, const string& str) {
 }
 
 
-/// move_to_san() takes a position and a move as input, where it is assumed
-/// that the move is a legal move for the position. The return value is
-/// a string containing the move in short algebraic notation.
+/// move_to_san() takes a position and a legal Move as input and returns its
+/// short algebraic notation representation.
 
 const string move_to_san(Position& pos, Move m) {
 
@@ -78,17 +79,17 @@ const string move_to_san(Position& pos, Move m) {
   if (m == MOVE_NULL)
       return "(null)";
 
-  assert(is_ok(m));
+  assert(pos.move_is_legal(m));
 
   Bitboard attackers;
   bool ambiguousMove, ambiguousFile, ambiguousRank;
-  Square sq, from = from_sq(m);
+  string san;
+  Square from = from_sq(m);
   Square to = to_sq(m);
   PieceType pt = type_of(pos.piece_on(from));
-  string san;
 
   if (is_castle(m))
-      san = (to_sq(m) < from_sq(m) ? "O-O-O" : "O-O");
+      san = to_sq(m) < from_sq(m) ? "O-O-O" : "O-O";
   else
   {
       if (pt != PAWN)
@@ -97,24 +98,20 @@ const string move_to_san(Position& pos, Move m) {
 
           // Disambiguation if we have more then one piece with destination 'to'
           // note that for pawns is not needed because starting file is explicit.
-          attackers = pos.attackers_to(to) & pos.pieces(pt, pos.side_to_move());
-          clear_bit(&attackers, from);
+          attackers = pos.attackers_to(to) & pos.pieces(pos.side_to_move(), pt);
+          attackers ^= from;
           ambiguousMove = ambiguousFile = ambiguousRank = false;
 
           while (attackers)
           {
-              sq = pop_1st_bit(&attackers);
+              Square sq = pop_1st_bit(&attackers);
 
               // Pinned pieces are not included in the possible sub-set
               if (!pos.pl_move_is_legal(make_move(sq, to), pos.pinned_pieces()))
                   continue;
 
-              if (file_of(sq) == file_of(from))
-                  ambiguousFile = true;
-
-              if (rank_of(sq) == rank_of(from))
-                  ambiguousRank = true;
-
+              ambiguousFile |= file_of(sq) == file_of(from);
+              ambiguousRank |= rank_of(sq) == rank_of(from);
               ambiguousMove = true;
           }
 
@@ -122,8 +119,10 @@ const string move_to_san(Position& pos, Move m) {
           {
               if (!ambiguousFile)
                   san += file_to_char(file_of(from));
+
               else if (!ambiguousRank)
                   san += rank_to_char(rank_of(from));
+
               else
                   san += square_to_string(from);
           }
@@ -140,10 +139,7 @@ const string move_to_san(Position& pos, Move m) {
       san += square_to_string(to);
 
       if (is_promotion(m))
-      {
-          san += '=';
-          san += piece_type_to_char(promotion_piece_type(m));
-      }
+          san += string("=") + piece_type_to_char(promotion_type(m));
   }
 
   if (pos.move_gives_check(m, CheckInfo(pos)))