From: Maxim Masiutin <maxim@masiutin.com>
Date: Sat, 1 Apr 2023 20:49:03 +0000 (+0300)
Subject: Do no initialize TM in all cases
X-Git-Url: https://git.sesse.net/?p=stockfish;a=commitdiff_plain;h=1a64afb1c65591ccd374504c791eb27b762f6c8f

Do no initialize TM in all cases

Avoid doing full TM initialization if it won't be used, avoids division by zero.

closes https://github.com/official-stockfish/Stockfish/pull/4484

No functional change
---

diff --git a/src/timeman.cpp b/src/timeman.cpp
index 5c826b4f..061de018 100644
--- a/src/timeman.cpp
+++ b/src/timeman.cpp
@@ -36,6 +36,12 @@ TimeManagement Time; // Our global time management object
 
 void TimeManagement::init(Search::LimitsType& limits, Color us, int ply) {
 
+  // if we have no time, no need to initialize TM, except for the start time,
+  // which is used by movetime.
+  startTime = limits.startTime;
+  if (limits.time[us] == 0)
+      return;
+
   TimePoint moveOverhead    = TimePoint(Options["Move Overhead"]);
   TimePoint slowMover       = TimePoint(Options["Slow Mover"]);
   TimePoint npmsec          = TimePoint(Options["nodestime"]);
@@ -59,8 +65,6 @@ void TimeManagement::init(Search::LimitsType& limits, Color us, int ply) {
       limits.npmsec = npmsec;
   }
 
-  startTime = limits.startTime;
-
   // Maximum move horizon of 50 moves
   int mtg = limits.movestogo ? std::min(limits.movestogo, 50) : 50;