From: Marco Costalba <mcostalba@gmail.com>
Date: Thu, 18 Sep 2014 13:44:38 +0000 (+0200)
Subject: Rename time variable to reflect UCI parameters
X-Git-Url: https://git.sesse.net/?p=stockfish;a=commitdiff_plain;h=4d0a6c5a6f80fbb417318d6b5b22c5aab23e6de4;hp=2eec710318255df054a77abb7cd48d680453fd26;ds=sidebyside

Rename time variable to reflect UCI parameters

On top of previous patch, rename time variables to
reflect the simplification of UCI parameters.

It is more correct to use as varibales directly the
corresponding UCI option, without intorducing redundant
intermediate variables.

This allows also to simplify the code.

No functional change.
---

diff --git a/src/timeman.cpp b/src/timeman.cpp
index 11405bb9..88a52bbd 100644
--- a/src/timeman.cpp
+++ b/src/timeman.cpp
@@ -89,11 +89,9 @@ void TimeManager::init(const Search::LimitsType& limits, int currentPly, Color u
   int hypMTG, hypMyTime, t1, t2;
 
   // Read uci parameters
-  int emergencyMoveHorizon = 40;
-  int emergencyBaseTime    = Options["Move Overhead"] * 2;
-  int emergencyMoveTime    = Options["Move Overhead"];
-  int minThinkingTime      = Options["Minimum Thinking Time"];
-  int slowMover            = Options["Slow Mover"];
+  int moveOverhead    = Options["Move Overhead"];
+  int minThinkingTime = Options["Minimum Thinking Time"];
+  int slowMover       = Options["Slow Mover"];
 
   // Initialize unstablePvFactor to 1 and search times to maximum values
   unstablePvFactor = 1;
@@ -106,8 +104,7 @@ void TimeManager::init(const Search::LimitsType& limits, int currentPly, Color u
       // Calculate thinking time for hypothetical "moves to go"-value
       hypMyTime =  limits.time[us]
                  + limits.inc[us] * (hypMTG - 1)
-                 - emergencyBaseTime
-                 - emergencyMoveTime * std::min(hypMTG, emergencyMoveHorizon);
+                 - moveOverhead * (2 + std::min(hypMTG, 40));
 
       hypMyTime = std::max(hypMyTime, 0);