From 47b967aa38dcb5db60229931abf391f42c2abf1f Mon Sep 17 00:00:00 2001
From: H.G.Muller <hgm@hgm-xboard.(none)>
Date: Sat, 8 Dec 2018 09:54:37 +0100
Subject: [PATCH] Let 'searching' also indicate thinking state

The flag 'think'  is eliminated, and its function is now indicated by
giving 'searching' the value 3. This makes 'searching' now a complete
state indicator, which indicates if we are idle, pondering, analysing
or thinking.
---
 UCI2WB.c |   17 +++++++++--------
 1 files changed, 9 insertions(+), 8 deletions(-)

diff --git a/UCI2WB.c b/UCI2WB.c
index ae6cf2c..1fc24fc 100644
--- a/UCI2WB.c
+++ b/UCI2WB.c
@@ -446,7 +446,7 @@ GUI2Engine()
     char line[256], command[256], *p;
 
     while(1) {
-	int i, difficult, think=0;
+	int i, difficult;
 
 	if((computer == stm || computer == ANALYZE && !searching) && !suspended) {
 	    DPRINT("# start search\n");
@@ -460,14 +460,14 @@ GUI2Engine()
 		    for(i=1; i<nr; i++) if(on[i]) EPRINT((f, " %s", moveMap[i]))
 		}
 		EPRINT((f, "\n")) searching = 2; // suppresses spurious commands during analysis starting new searches
-	    } else pause = think = 2, StartSearch(""); // request suspending of input processing while thinking
+	    } else pause = 2, searching = 3, StartSearch(""); // request suspending of input processing while thinking
 	} else if(ponderAlways && computer == NONE) move[moveNr][0] = 0, StartPonder(moveNr-1);
       nomove:
        for(difficult=0; !difficult; ) { // read and handle commands that can (or must) be handled during thinking
 	fflush(toE); fflush(stdout);
 	if(!ReadLine(stdin, line)) printf("# EOF\n"), sprintf(line, "quit -1\n");
-	if(think && !pause) Sync(PAUSE), think = 0, Release(); // if no longer thinking, take dummy pause
-	sscanf(line, "%s", command); DPRINT("# '%s' think=%d pause=%d log=%d sent=%d\n", command, think, pause, logLen, sentLen);
+	if(searching == 3 && !pause) Sync(PAUSE), searching = 0, Release(); // if no longer thinking, take dummy pause
+	sscanf(line, "%s", command); DPRINT("# '%s' searching=%d pause=%d log=%d sent=%d\n", command, searching, pause, logLen, sentLen);
 	if(!strcmp(command, "usermove")) { difficult--; break; } // for efficiency during game play, moves, time & otim are tried first
 	else if(!strcmp(command, "time"))   sscanf(line+4, "%d", &myTime),  myTime  = (10*myTime)/unit;
 	else if(!strcmp(command, "otim"))   sscanf(line+4, "%d", &hisTime), hisTime = (10*hisTime)/unit;
@@ -499,7 +499,7 @@ GUI2Engine()
 		snprintf(backLog+logLen, 9999-logLen, "setoption %s%s %s%s", nameWord, line+7, valueWord, p);
 	    } else snprintf(backLog+logLen, 9999-logLen, "setoption %s%s\n", nameWord, line+7);
 	    DPRINT("# backlog: %s", backLog+logLen); logLen += strlen(backLog+logLen);
-	    if(!think && Release()) break; // break will restart analysis; pondering is restarted by Release itself
+	    if(searching < 3 && Release()) break; // break will restart analysis; pondering is restarted by Release itself
 	  }
 	  else difficult = 1; // difficult command; terminate loop for easy ones
 	}
@@ -516,7 +516,7 @@ GUI2Engine()
 		    char *draw = drawOffer ? " draw" : ""; drawOffer = 0;
 		    searching = 0; pause = 2; moveNr++; startTime = GetTickCount(); // clock starts running now
 		    EPRINT((f, "# ponderhit%s\n", draw)) fflush(toE); fflush(stdout);
-		    think = 2; // request blocking input during thinking
+		    searching = 3; // request blocking input during thinking
 		    goto nomove;
 		}
 		StopPonder(1); searching = 0;
@@ -525,10 +525,11 @@ GUI2Engine()
 	    continue;
 	}
 	if(!strcmp(command, "resume")) {
+	    searching = 0;
 	    if(suspended == 2) StartPonder(moveNr); // restart interrupted ponder search
-	    suspended = think = 0; continue;  // causes thinking to start in normal way if on move or analyzing
+	    suspended = 0; continue;  // causes thinking to start in normal way if on move or analyzing
 	}
-	if(think) { // command arrived during thinking; order abort for 'instant commands'
+	if(searching == 3) { // command arrived during thinking; order abort for 'instant commands'
 	    if(!strcmp(command, "?") || !strcmp(command, "quit") ||
 	       !strcmp(command, "force") || !strcmp(command, "result")) { EPRINT((f, "# stop\n")); fflush(toE); }
 	    Sync(PAUSE); Release(); // block processing of difficult commands during thinking; send backlog left because of race
-- 
1.7.0.4