X-Git-Url: https://git.sesse.net/?p=stockfish;a=blobdiff_plain;f=src%2Fuci.cpp;h=34a26586a6a8295f0617b22d4945bfc16dc6af65;hp=ae34043831cb19f39fa96fd7c9f677f7d68733a0;hb=09d01ee9dc16f20962d5f5ffcb61ade56d4fa579;hpb=1e7aaed8bc4247a742d515811f0e484ff40309b8

diff --git a/src/uci.cpp b/src/uci.cpp
index ae340438..34a26586 100644
--- a/src/uci.cpp
+++ b/src/uci.cpp
@@ -33,7 +33,6 @@
 #include "move.h"
 #include "movegen.h"
 #include "position.h"
-#include "san.h"
 #include "search.h"
 #include "ucioption.h"
 
@@ -54,7 +53,6 @@ namespace {
   void set_position(Position& pos, UCIParser& up);
   bool go(Position& pos, UCIParser& up);
   void perft(Position& pos, UCIParser& up);
-  Move parse_uci_move(const Position& pos, const std::string &str);
 }
 
 
@@ -69,22 +67,20 @@ bool execute_uci_command(const string& cmd) {
   UCIParser up(cmd);
   string token;
 
-  if (!(up >> token)) // operator>>() skips any whitespace
-      return true;
+  up >> token; // operator>>() skips any whitespace
 
   if (token == "quit")
       return false;
 
-  if (token == "go")
+  else if (token == "go")
       return go(pos, up);
 
-  if (token == "uci")
-  {
+  else if (token == "uci")
       cout << "id name " << engine_name()
-           << "\nid author Tord Romstad, Marco Costalba, Joona Kiiski\n";
-      print_uci_options();
-      cout << "uciok" << endl;
-  }
+           << "\nid author " << engine_authors()
+           << "\n" << Options.print_all()
+           << "\nuciok" << endl;
+
   else if (token == "ucinewgame")
       pos.from_fen(StartPositionFEN, false);
 
@@ -97,30 +93,28 @@ bool execute_uci_command(const string& cmd) {
   else if (token == "setoption")
       set_option(up);
 
-  // The remaining commands are for debugging purposes only
   else if (token == "d")
       pos.print();
 
-  else if (token == "flip")
-  {
-      Position p(pos, pos.thread());
-      pos.flipped_copy(p);
-  }
   else if (token == "eval")
   {
-      Value evalMargin;
-      cout << "Incremental mg: "   << mg_value(pos.value())
-           << "\nIncremental eg: " << eg_value(pos.value())
-           << "\nFull eval: "      << evaluate(pos, evalMargin) << endl;
+      read_evaluation_uci_options(pos.side_to_move());
+      cout << trace_evaluate(pos) << endl;
   }
+
   else if (token == "key")
-      cout << "key: " << hex << pos.get_key()
+      cout << "key: " << hex     << pos.get_key()
            << "\nmaterial key: " << pos.get_material_key()
-           << "\npawn key: " << pos.get_pawn_key() << endl;
+           << "\npawn key: "     << pos.get_pawn_key() << endl;
 
   else if (token == "perft")
       perft(pos, up);
 
+  else if (token == "flip")
+  {
+      Position p(pos, pos.thread());
+      pos.flipped_copy(p);
+  }
   else
       cout << "Unknown command: " << cmd << endl;
 
@@ -134,75 +128,6 @@ bool execute_uci_command(const string& cmd) {
 
 namespace {
 
-  // parse_uci_move() takes a position and a string as input, and attempts to
-  // convert the string to a move, using simple coordinate notation (g1f3,
-  // a7a8q, etc.). In order to correctly parse en passant captures and castling
-  // moves, we need the position. This function is not robust, and expects that
-  // the input move is legal and correctly formatted.
-
-  Move parse_uci_move(const Position& pos, const std::string& str) {
-
-    Square from, to;
-    Piece piece;
-    Color us = pos.side_to_move();
-
-    if (str.length() < 4)
-        return MOVE_NONE;
-
-    // Read the from and to squares
-    from = make_square(file_from_char(str[0]), rank_from_char(str[1]));
-    to   = make_square(file_from_char(str[2]), rank_from_char(str[3]));
-
-    // Find the moving piece
-    piece = pos.piece_on(from);
-
-    // If the string has more than 4 characters, try to interpret the 5th
-    // character as a promotion.
-    if (str.length() > 4 && piece == piece_of_color_and_type(us, PAWN))
-    {
-        switch (tolower(str[4])) {
-        case 'n':
-            return make_promotion_move(from, to, KNIGHT);
-        case 'b':
-            return make_promotion_move(from, to, BISHOP);
-        case 'r':
-            return make_promotion_move(from, to, ROOK);
-        case 'q':
-            return make_promotion_move(from, to, QUEEN);
-        }
-    }
-
-    // En passant move? We assume that a pawn move is an en passant move
-    // if the destination square is epSquare.
-    if (to == pos.ep_square() && piece == piece_of_color_and_type(us, PAWN))
-        return make_ep_move(from, to);
-
-    // Is this a castling move? A king move is assumed to be a castling move
-    // if the destination square is occupied by a friendly rook, or if the
-    // distance between the source and destination squares is more than 1.
-    if (piece == piece_of_color_and_type(us, KING))
-    {
-        if (pos.piece_on(to) == piece_of_color_and_type(us, ROOK))
-            return make_castle_move(from, to);
-
-        if (square_distance(from, to) > 1)
-        {
-            // This is a castling move, but we have to translate it to the
-            // internal "king captures rook" representation.
-            SquareDelta delta = (to > from ? DELTA_E : DELTA_W);
-            Square s = from;
-
-            do s += delta;
-            while (   pos.piece_on(s) != piece_of_color_and_type(us, ROOK)
-                   && relative_rank(us, s) == RANK_1);
-
-            return relative_rank(us, s) == RANK_1 ? make_castle_move(from, s) : MOVE_NONE;
-        }
-    }
-
-    return make_move(from, to);
-  }
-
   // set_position() is called when Stockfish receives the "position" UCI
   // command. The input parameter is a UCIParser. It is assumed
   // that this parser has consumed the first token of the UCI command
@@ -211,42 +136,27 @@ namespace {
 
   void set_position(Position& pos, UCIParser& up) {
 
-    string token;
+    string fen, token;
 
-    if (!(up >> token) || (token != "startpos" && token != "fen"))
-        return;
+    up >> token; // operator>>() skips any whitespace
 
     if (token == "startpos")
     {
         pos.from_fen(StartPositionFEN, false);
-        if (!(up >> token))
-            return;
+        up >> token; // Consume "moves" token
     }
-    else // fen
+    else if (token == "fen")
     {
-        string fen;
         while (up >> token && token != "moves")
-        {
-            fen += token;
-            fen += ' ';
-        }
+            fen += token + " ";
+
         pos.from_fen(fen, Options["UCI_Chess960"].value<bool>());
     }
+    else return;
 
-    if (token != "moves")
-        return;
-
-    // Parse optional move list
-    Move move;
-    StateInfo st;
+    // Parse move list (if any)
     while (up >> token)
-    {
-        move = parse_uci_move(pos, token);
-        pos.do_setup_move(move, st);
-    }
-    // Our StateInfo st is about going out of scope so copy
-    // its content inside pos before it disappears.
-    pos.detach();
+        pos.do_setup_move(move_from_uci(pos, token));
   }
 
 
@@ -258,39 +168,26 @@ namespace {
 
   void set_option(UCIParser& up) {
 
-    string token, name, value;
-
-    if (!(up >> token) || token != "name") // operator>>() skips any whitespace
-        return;
+    string value = "true"; // UCI buttons don't have a "value" field
+    string token, name;
 
-    if (!(up >> name))
-        return;
+    up >> token; // Consume "name" token
+    up >> name;  // Read option name
 
     // Handle names with included spaces
     while (up >> token && token != "value")
-        name += (" " + token);
+        name += " " + token;
 
-    if (Options.find(name) == Options.end())
-    {
-        cout << "No such option: " << name << endl;
-        return;
-    }
-
-    // Is a button ?
-    if (token != "value")
-    {
-        Options[name].set_value("true");
-        return;
-    }
-
-    if (!(up >> value))
-        return;
+    up >> value; // Read option value
 
     // Handle values with included spaces
     while (up >> token)
-        value += (" " + token);
+        value += " " + token;
 
-    Options[name].set_value(value);
+    if (Options.find(name) != Options.end())
+        Options[name].set_value(value);
+    else
+        cout << "No such option: " << name << endl;
   }
 
 
@@ -306,50 +203,48 @@ namespace {
   bool go(Position& pos, UCIParser& up) {
 
     string token;
-
-    int time[2] = {0, 0}, inc[2] = {0, 0};
-    int movesToGo = 0, depth = 0, nodes = 0, moveTime = 0;
-    bool infinite = false, ponder = false;
-    Move searchMoves[MOVES_MAX];
-
-    searchMoves[0] = MOVE_NONE;
+    SearchLimits limits;
+    Move searchMoves[MAX_MOVES] = { MOVE_NONE };
+    Move* cur = searchMoves;
+    int time[] = { 0, 0 }, inc[] = { 0, 0 };
 
     while (up >> token)
     {
         if (token == "infinite")
-            infinite = true;
+            limits.infinite = true;
         else if (token == "ponder")
-            ponder = true;
+            limits.ponder = true;
         else if (token == "wtime")
-            up >> time[0];
+            up >> time[WHITE];
         else if (token == "btime")
-            up >> time[1];
+            up >> time[BLACK];
         else if (token == "winc")
-            up >> inc[0];
+            up >> inc[WHITE];
         else if (token == "binc")
-            up >> inc[1];
+            up >> inc[BLACK];
         else if (token == "movestogo")
-            up >> movesToGo;
+            up >> limits.movesToGo;
         else if (token == "depth")
-            up >> depth;
+            up >> limits.maxDepth;
         else if (token == "nodes")
-            up >> nodes;
+            up >> limits.maxNodes;
         else if (token == "movetime")
-            up >> moveTime;
+            up >> limits.maxTime;
         else if (token == "searchmoves")
         {
-            int numOfMoves = 0;
             while (up >> token)
-                searchMoves[numOfMoves++] = parse_uci_move(pos, token);
+                *cur++ = move_from_uci(pos, token);
 
-            searchMoves[numOfMoves] = MOVE_NONE;
+            *cur = MOVE_NONE;
         }
     }
 
     assert(pos.is_ok());
 
-    return think(pos, infinite, ponder, time, inc, movesToGo,
-                 depth, nodes, moveTime, searchMoves);
+    limits.time = time[pos.side_to_move()];
+    limits.increment = inc[pos.side_to_move()];
+
+    return think(pos, limits, searchMoves);
   }
 
   void perft(Position& pos, UCIParser& up) {