Cleanup code

This PR includes following cleanups:
- Remove the unused depth variable in the thread class.
- cleanup ValueList (added from mstembera)

closes https://github.com/official-stockfish/Stockfish/pull/4127

No functional change.

diff --git a/src/evaluate.cpp b/src/evaluate.cpp
index 9e3eaba512c5e1de107d9e2f438f1a0e954abd80..7d5876758c0836e5ad43e363e4dd9f5f719502aa 100644 (file)
--- a/src/evaluate.cpp
+++ b/src/evaluate.cpp
@@ -1114,7 +1114,6 @@ std::string Eval::trace(Position& pos) {
   std::memset(scores, 0, sizeof(scores));
 
   // Reset any global variable used in eval
-  pos.this_thread()->depth           = 0;
   pos.this_thread()->trend           = SCORE_ZERO;
   pos.this_thread()->bestValue       = VALUE_ZERO;
   pos.this_thread()->optimism[WHITE] = VALUE_ZERO;

diff --git a/src/misc.h b/src/misc.h
index 2fd2b408a1261644c7b0c7315f91dcb77cc2bf54..fe1143def0b35150eeedf51d3976baa46beb467e 100644 (file)
--- a/src/misc.h
+++ b/src/misc.h
@@ -116,23 +116,10 @@ class ValueList {
 
 public:
   std::size_t size() const { return size_; }
-  void resize(std::size_t newSize) { size_ = newSize; }
   void push_back(const T& value) { values_[size_++] = value; }
-  T& operator[](std::size_t index) { return values_[index]; }
-  T* begin() { return values_; }
-  T* end() { return values_ + size_; }
-  const T& operator[](std::size_t index) const { return values_[index]; }
   const T* begin() const { return values_; }
   const T* end() const { return values_ + size_; }
 
-  void swap(ValueList& other) {
-    const std::size_t maxSize = std::max(size_, other.size_);
-    for (std::size_t i = 0; i < maxSize; ++i) {
-      std::swap(values_[i], other.values_[i]);
-    }
-    std::swap(size_, other.size_);
-  }
-
 private:
   T values_[MaxSize];
   std::size_t size_ = 0;

diff --git a/src/position.cpp b/src/position.cpp
index ec9229ea33799e2b0234093912612d9d704d6845..08ed1a89e1578f8202392c4bbf61daa8653fcda9 100644 (file)
--- a/src/position.cpp
+++ b/src/position.cpp
@@ -1099,10 +1099,12 @@ bool Position::see_ge(Move m, Value threshold) const {
       // Don't allow pinned pieces to attack as long as there are
       // pinners on their original square.
       if (pinners(~stm) & occupied)
+      {
           stmAttackers &= ~blockers_for_king(stm);
 
-      if (!stmAttackers)
-          break;
+          if (!stmAttackers)
+              break;
+      }
 
       res ^= 1;
 

diff --git a/src/search.cpp b/src/search.cpp
index 7c0601e60e2b2f1ab2c3ce83c8d2641bc8743d1d..565fba0f89f7ece8ffccaf1aef5ea00beff5b3b6 100644 (file)
--- a/src/search.cpp
+++ b/src/search.cpp
@@ -565,7 +565,6 @@ namespace {
 
     // Step 1. Initialize node
     Thread* thisThread = pos.this_thread();
-    thisThread->depth  = depth;
     ss->inCheck        = pos.checkers();
     priorCapture       = pos.captured_piece();
     Color us           = pos.side_to_move();

diff --git a/src/thread.h b/src/thread.h
index 9e9cd488c3c2fa71d9c987bde2043059e4d453cc..c430a8184e6da110a5addac0bbabd57acdcb3618 100644 (file)
--- a/src/thread.h
+++ b/src/thread.h
@@ -69,7 +69,7 @@ public:
   Position rootPos;
   StateInfo rootState;
   Search::RootMoves rootMoves;
-  Depth rootDepth, completedDepth, depth, previousDepth;
+  Depth rootDepth, completedDepth, previousDepth;
   Value rootDelta;
   CounterMoveHistory counterMoves;
   ButterflyHistory mainHistory;