Fix pause/resume
authorH.G.Muller <hgm@hgm-xboard.(none)>
Sat, 8 Dec 2018 11:59:02 +0000 (12:59 +0100)
committerH.G.Muller <hgm@hgm-xboard.(none)>
Sat, 8 Dec 2018 11:59:02 +0000 (12:59 +0100)
Tne 'pause' and 'resume' commands are executed in the GUI thread for
instant obedience. But they have to take care that a move produced by
an aborted think is ignored (by setting searching = 0 before stopping it),
and that the engine thread has to be signalled to restart it.

UCI2WB.c

index 536eb44..b901bbc 100644 (file)
--- a/UCI2WB.c
+++ b/UCI2WB.c
@@ -460,7 +460,7 @@ GUI2Engine()
        else if(!strcmp(command, "pause")) {\r
            if(computer == stm) myTime -= GetTickCount() - startTime;\r
            suspended = 1 + (searching == 1); // remember if we were pondering, and stop search ignoring bestmove\r
-           StopPonder(searching);\r
+           if(searching) searching = 0, StopPonder(1);\r
        }\r
        else { //convert easy & hard to "option" after treating their effect on the adapter\r
          if(!strcmp(command, "easy")) {\r
@@ -468,8 +468,8 @@ GUI2Engine()
          }\r
          else if(!strcmp(command, "hard")) {\r
            if(*canPonder) ponder = 1, sprintf(command, "option"), sprintf(line, "option %s=1\n", canPonder); else continue;\r
-         } else\r
-         if(!strcmp(command, "option")) {\r
+         }\r
+         else if(!strcmp(command, "option")) {\r
            if(sscanf(line+7, "UCI2WB debug output=%d", &debug) == 1) ; else\r
            if(sscanf(line+7, "ponder always=%d", &ponderAlways) == 1) ; else\r
            if(sscanf(line+7, "Floating Byoyomi=%d", &flob) == 1) ; else\r
@@ -499,9 +499,8 @@ GUI2Engine()
            *qEnd++ = '\n'; Sync(WAKEUP);    // make sure engine thread considers starting a search\r
        } else\r
        if(!strcmp(command, "resume")) {\r
-           searching = 0;\r
            if(suspended == 2) StartPonder(moveNr); // restart interrupted ponder search\r
-           suspended = 0;  // causes thinking to start in normal way if on move or analyzing\r
+           suspended = 0;  *qEnd++ = '\n'; Sync(WAKEUP); // causes search to start in normal way if on move or analyzing\r
        } else\r
       {\r
        if(searching == 3) { // command arrived during thinking; order abort for 'instant commands'\r