From: loco-loco <loco-loco@users.noreply.github.com>
Date: Thu, 12 Jan 2017 07:46:46 +0000 (-0800)
Subject: Removing CM parameter from the Stats struct. (#970)
X-Git-Url: https://git.sesse.net/?p=stockfish;a=commitdiff_plain;h=99cd513264be1ad61848bbefd678be0feaa08061;hp=d2971f3fca929d9085ed17b71aa4ec8e96499d99

Removing CM parameter from the Stats struct. (#970)

After the history simplifications, we are only using Value Stats for CounterMoveHistory table. Therefore the parameter CM is not necessary.

No functional change.
---

diff --git a/src/movepick.h b/src/movepick.h
index ec8b1554..0ef2c89b 100644
--- a/src/movepick.h
+++ b/src/movepick.h
@@ -60,7 +60,7 @@ private:
 /// Entries are stored using only the moving piece and destination square, hence
 /// two moves with different origin but same destination and piece will be
 /// considered identical.
-template<typename T, bool CM = false>
+template<typename T>
 struct Stats {
   const T* operator[](Piece pc) const { return table[pc]; }
   T* operator[](Piece pc) { return table[pc]; }
@@ -71,7 +71,7 @@ struct Stats {
     if (abs(int(v)) >= 324)
         return;
 
-    table[pc][to] -= table[pc][to] * abs(int(v)) / (CM ? 936 : 324);
+    table[pc][to] -= table[pc][to] * abs(int(v)) / 936;
     table[pc][to] += int(v) * 32;
   }
 
@@ -80,7 +80,7 @@ private:
 };
 
 typedef Stats<Move> MoveStats;
-typedef Stats<Value, true> CounterMoveStats;
+typedef Stats<Value> CounterMoveStats;
 typedef Stats<CounterMoveStats> CounterMoveHistoryStats;