fixed some user bugs

Author: winapiadmin

main.cpp

@@ -2,19 +2,25 @@
 #include "uci.h"
 #include "ucioption.h"
 #include <iostream>
+#include <thread>
 using namespace engine;
+extern std::thread searchThread;
 int main() {
   options.add("Move Overhead", Option(10, 0, 1000));
-  options.add( //
-      "Hash", Option(16, 1, 1 << 25, [](const Option &o) {
-        search::tt.resize((o.operator int()));
-        return std::nullopt;
-      }));
+  options.add("Hash", Option(16, 1, 1 << 25, [](const Option &o) {
+                search::tt.resize((o.operator int()));
+                return std::nullopt;
+              }));
 
-  options.add( //
-      "Clear Hash", Option(+[](const Option &) {
-        search::tt.clear();
-        return std::nullopt;
-      }));
+  options.add("Clear Hash", Option(+[](const Option &) {
+                if (searchThread.joinable()) {
+                  std::cout
+                      << "info string In search, do not modify hash table\n";
+                  return std::nullopt;
+                }
+                search::tt.clear();
+
+                return std::nullopt;
+              }));
   loop();
-}
+}

uci.cpp

@@ -11,7 +11,12 @@
 using namespace engine;
 chess::Position pos;
 OptionsMap engine::options;
+std::thread searchThread;
 void handlePosition(std::istringstream &is) {
+  if (searchThread.joinable()) {
+    std::cout << "info string In search, do not modify position\n";
+    return;
+  }
   std::string token, fen;
 
   is >> token;
@@ -71,7 +76,6 @@ timeman::LimitsType parse_limits(std::istream &is) {
 
   return limits;
 }
-std::thread searchThread;
 
 void handleGo(std::istringstream &ss) {
   if (searchThread.joinable()) {
@@ -198,4 +202,7 @@ void engine::loop() {
       }
     }
   }
+  search::stop();
+  if (searchThread.joinable())
+    searchThread.join();
 }