Simplify increaseDepth to boolean expression

FauziAkram · Disservin · commit daa3ef9148e8 · 2024-03-12T16:43:10.000+01:00
Non-functional Simplification, maintaining the same logic as before. Big thanks to @peregrineshahin for helping with the code. Passed non-regression bounds: https://tests.stockfishchess.org/tests/view/65ec93860ec64f0526c42375 LLR: 2.93 (-2.94,2.94) <-1.75,0.25> Total: 101088 W: 26196 L: 26047 D: 48845 Ptnml(0-2): 405, 11580, 26473, 11633, 453 closes #5103 No functional change
diff --git a/src/search.cpp b/src/search.cpp
@@ -464,11 +464,10 @@ void Search::Worker::iterative_deepening() {
                 else
                     threads.stop = true;
             }
-            else if (!mainThread->ponder
-                     && mainThread->tm.elapsed(threads.nodes_searched()) > totalTime * 0.506)
-                threads.increaseDepth = false;
             else
-                threads.increaseDepth = true;
+                threads.increaseDepth =
+                  mainThread->ponder
+                  || mainThread->tm.elapsed(threads.nodes_searched()) <= totalTime * 0.506;
         }
 
         mainThread->iterValue[iterIdx] = bestValue;

Original file line number	Diff line number	Diff line change
`@@ -464,11 +464,10 @@ void Search::Worker::iterative_deepening() {`
`464`	`464`	`else`
`465`	`465`	`threads.stop = true;`
`466`	`466`	`}`
`467`		`- else if (!mainThread->ponder`
`468`		`- && mainThread->tm.elapsed(threads.nodes_searched()) > totalTime * 0.506)`
`469`		`- threads.increaseDepth = false;`
`470`	`467`	`else`
`471`		`- threads.increaseDepth = true;`
	`468`	`+ threads.increaseDepth =`
	`469`	`+ mainThread->ponder`
	`470`	`+ \|\| mainThread->tm.elapsed(threads.nodes_searched()) <= totalTime * 0.506;`
`472`	`471`	`}`
`473`	`472`
`474`	`473`	`mainThread->iterValue[iterIdx] = bestValue;`