Initialize futilityMargin in EvalInfo c'tor

mcostalba · mcostalba · commit b5d38ad1e5cc · 2010-01-17T13:24:40.000+01:00
This is less prone to bugs because now it's up to the
compiler don't forget this important initialization.

No functional change.

Signed-off-by: Marco Costalba &lt;mcostalba@gmail.com&gt;
diff --git a/src/evaluate.h b/src/evaluate.h
@@ -47,6 +47,8 @@ class Position;
 
 struct EvalInfo {
 
+  EvalInfo() : futilityMargin(Value(0)) {}
+
   // Middle game and endgame evaluations
   Score value;
 
diff --git a/src/search.cpp b/src/search.cpp
@@ -1370,7 +1370,6 @@ namespace {
     }
 
     isCheck = pos.is_check();
-    ei.futilityMargin = Value(0); // Manually initialize futilityMargin
 
     // Calculate depth dependant futility pruning parameters
     const int FutilityMoveCountMargin = 3 + (1 << (3 * int(depth) / 8));
@@ -1672,7 +1671,6 @@ namespace {
     }
 
     isCheck = pos.is_check();
-    ei.futilityMargin = Value(0); // Manually initialize futilityMargin
 
     // Evaluate the position statically
     if (isCheck)

Original file line number	Diff line number	Diff line change
`@@ -1370,7 +1370,6 @@ namespace {`
`1370`	`1370`	`}`
`1371`	`1371`
`1372`	`1372`	`isCheck = pos.is_check();`
`1373`		`- ei.futilityMargin = Value(0); // Manually initialize futilityMargin`
`1374`	`1373`
`1375`	`1374`	`// Calculate depth dependant futility pruning parameters`
`1376`	`1375`	`const int FutilityMoveCountMargin = 3 + (1 << (3 * int(depth) / 8));`
`@@ -1672,7 +1671,6 @@ namespace {`
`1672`	`1671`	`}`
`1673`	`1672`
`1674`	`1673`	`isCheck = pos.is_check();`
`1675`		`- ei.futilityMargin = Value(0); // Manually initialize futilityMargin`
`1676`	`1674`
`1677`	`1675`	`// Evaluate the position statically`
`1678`	`1676`	`if (isCheck)`