make timeLeft a double, timepoint seemed unecessary since it was always casting back to double anyway.

TierynnB · TierynnB · commit de4a3c47a666 · 2024-02-19T07:32:09.000+10:00
diff --git a/src/timeman.cpp b/src/timeman.cpp
@@ -84,10 +84,10 @@ void TimeManagement::init(Search::LimitsType& limits,
     // Maximum move horizon of 50 moves
     int mtg = limits.movestogo ? std::min(limits.movestogo, 50) : 50;
 
-    // Make sure timeLeft is > 0 since we may use it as a divisor
+    // Make sure timeLeft is > 0 since we use it as a divisor
     // If there is a healthy increment and low mtg, timeLeft can exceed actual available
-    TimePoint timeLeft = std::max(limits.time[us], limits.time[us] + limits.inc[us] * (mtg - 1)
-                                                     - moveOverhead * (2 + mtg));
+    double timeLeft =
+      std::max(limits.time[us], limits.time[us] + limits.inc[us] * (mtg - 1) - moveOverhead * mtg);
 
     // x basetime (+ z increment)
     // game time for the current move, so also cap to 20% of available game time.
@@ -102,15 +102,15 @@ void TimeManagement::init(Search::LimitsType& limits,
         double maxConstant = std::max(3.4 + 3.0 * std::log10(limits.time[us] / 1000.0), 2.76);
 
         optScale = std::min(0.0120 + std::pow(ply + 3.1, 0.44) * optConstant,
-                            0.21 * limits.time[us] / double(timeLeft))
+                            0.21 * limits.time[us] / timeLeft)
                  * optExtra;
         maxScale = std::min(6.9, maxConstant + ply / 12.2);
     }
 
     // x moves in y seconds (+ z increment)
     else
     {
-        optScale = std::min((0.88 + ply / 116.4) / mtg, 0.88 * limits.time[us] / double(timeLeft));
+        optScale = std::min((0.88 + ply / 116.4) / mtg, 0.88 * limits.time[us] / timeLeft);
         maxScale = std::min(6.3, 1.5 + 0.11 * mtg);
     }
 
