From: Joona Kiiski <joona.kiiski@gmail.com>
Date: Sun, 18 Jul 2010 20:52:03 +0000 (+0300)
Subject: Drop KILLER_MAX. Hardcode to 2 instead.
X-Git-Url: https://git.sesse.net/?p=stockfish;a=commitdiff_plain;h=66c5835080c8c1dab6bf203d2b6d9db6ab9439cc;hp=a6d13428f644d2ca475a190295f9528c92bdaaec

Drop KILLER_MAX. Hardcode to 2 instead.

KILLER_MAX in search.h is quite pointless, because
we already hardcode this to 2 in MovePicker anyway.

By hard-coding this to 2 we can keep code simpler.

No functional change

Signed-off-by: Marco Costalba <mcostalba@gmail.com>
---

diff --git a/src/search.cpp b/src/search.cpp
index 0f5e6907..7aab835d 100644
--- a/src/search.cpp
+++ b/src/search.cpp
@@ -368,9 +368,7 @@ void SearchStack::init() {
 // SearchStack::initKillers() initializes killers for a search stack entry
 void SearchStack::initKillers() {
 
-  mateKiller = MOVE_NONE;
-  for (int i = 0; i < KILLER_MAX; i++)
-      killers[i] = MOVE_NONE;
+  killers[0] = killers[1] = mateKiller = MOVE_NONE;
 }
 
 
@@ -1872,10 +1870,8 @@ namespace {
 
   bool move_is_killer(Move m, SearchStack* ss) {
 
-      const Move* k = ss->killers;
-      for (int i = 0; i < KILLER_MAX; i++, k++)
-          if (*k == m)
-              return true;
+      if (ss->killers[0] == m || ss->killers[1] == m)
+          return true;
 
       return false;
   }
@@ -2052,9 +2048,7 @@ namespace {
     if (m == ss->killers[0])
         return;
 
-    for (int i = KILLER_MAX - 1; i > 0; i--)
-        ss->killers[i] = ss->killers[i - 1];
-
+    ss->killers[1] = ss->killers[0];
     ss->killers[0] = m;
   }
 
diff --git a/src/search.h b/src/search.h
index 61a33242..74951548 100644
--- a/src/search.h
+++ b/src/search.h
@@ -36,7 +36,6 @@
 
 const int PLY_MAX = 100;
 const int PLY_MAX_PLUS_2 = 102;
-const int KILLER_MAX = 2;
 
 
 ////
@@ -55,7 +54,7 @@ struct SearchStack {
   Move threatMove;
   Move excludedMove;
   Move bestMove;
-  Move killers[KILLER_MAX];
+  Move killers[2];
   Depth reduction;
   Value eval;
   bool skipNullMove;