From 9e92f9f1e4e601912bd40334c9f89677b9b65f23 Mon Sep 17 00:00:00 2001 From: H.G.Muller Date: Sat, 8 Dec 2018 10:21:03 +0100 Subject: [PATCH] Launch searches at end of GUI loop The loop for interpreting GUI commands is restructured by launching searches at the bottom of it rather than at the top. As it was an infinite loop anyway this only eliminates the first call to LaunchSearch(), which is OK as the engine would never have to search before any commands were received. As this change brought the 'nomove' label to the top of the loop, the 'goto's to it could be replaced by 'continue's, and the label deleted. That for engines that cannot ponder the 'easy' and 'hard' commands now skip LaunchSearch() is OK, as LaunchSearch() would not do anything in that case. --- UCI2WB.c | 8 ++++---- 1 files changed, 4 insertions(+), 4 deletions(-) diff --git a/UCI2WB.c b/UCI2WB.c index d44de74..b9df43c 100644 --- a/UCI2WB.c +++ b/UCI2WB.c @@ -469,8 +469,6 @@ GUI2Engine() while(1) { int i, difficult; - LaunchSearch(); // start a search if we need one - 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"); @@ -525,7 +523,7 @@ GUI2Engine() searching = 0; pause = 2; moveNr++; startTime = GetTickCount(); // clock starts running now EPRINT((f, "# ponderhit%s\n", draw)) fflush(toE); fflush(stdout); searching = 3; // request blocking input during thinking - goto nomove; + continue; } StopPonder(1); searching = 0; } @@ -544,8 +542,10 @@ GUI2Engine() } if(qStart == qEnd) qStart = qEnd = queue; p = line; while(qEnd < queue+10000 && (*qEnd++ = *p++) != '\n') {} - if(DoCommand()) goto nomove; + if(DoCommand()) continue; } + + LaunchSearch(); // start a search if we need one } } -- 1.7.0.4