Use DEPTH_MAX instead of MAX_PLY
authorMarco Costalba <mcostalba@gmail.com>
Sat, 15 Nov 2014 04:36:49 +0000 (05:36 +0100)
committerMarco Costalba <mcostalba@gmail.com>
Sat, 15 Nov 2014 04:36:49 +0000 (05:36 +0100)
When comparing to a Depth it is more
consistent to use DEPTH_MAX instead
of a int.

This is a subtle difference because we use
ply and depth almost interchangably in SF,
but they are different. FOr counting plies
makes ense to continue using ints, while
for Depth we have our specific enum.

This cleanly fixes a new Clang 3.5 warning:

No functional change.

src/search.cpp
src/types.h

index 376705d..ccea71c 100644 (file)
@@ -255,7 +255,7 @@ namespace {
     multiPV = std::max(multiPV, skill.candidates_size());
 
     // Iterative deepening loop until requested to stop or target depth reached
-    while (++depth < MAX_PLY && !Signals.stop && (!Limits.depth || depth <= Limits.depth))
+    while (++depth < DEPTH_MAX && !Signals.stop && (!Limits.depth || depth <= Limits.depth))
     {
         // Age out PV variability metric
         BestMoveChanges *= 0.5;
index 94be833..d725c13 100644 (file)
@@ -212,7 +212,8 @@ enum Depth {
   DEPTH_QS_NO_CHECKS  = -1,
   DEPTH_QS_RECAPTURES = -5,
 
-  DEPTH_NONE = -6
+  DEPTH_NONE = -6,
+  DEPTH_MAX  = MAX_PLY
 };
 
 enum Square {