From c2c0e6b07de96c1fc39528b613fb3c9f93a8d7dc Mon Sep 17 00:00:00 2001
From: ElbertoOne <ElbertoOne@users.noreply.github.com>
Date: Thu, 23 Jun 2016 16:14:51 +0200
Subject: [PATCH] Remove redundant PvNode condition

After commit 6d58bf777caa323 we always call PvNodes
with cutNode set to false.

No functional change.
---
 src/search.cpp | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/src/search.cpp b/src/search.cpp
index c98c3334..a36601bf 100644
--- a/src/search.cpp
+++ b/src/search.cpp
@@ -549,6 +549,7 @@ namespace {
     assert(-VALUE_INFINITE <= alpha && alpha < beta && beta <= VALUE_INFINITE);
     assert(PvNode || (alpha == beta - 1));
     assert(DEPTH_ZERO < depth && depth < DEPTH_MAX);
+    assert(!(PvNode && cutNode));
 
     Move pv[MAX_PLY+1], quietsSearched[64];
     StateInfo st;
@@ -963,7 +964,7 @@ moves_loop: // When in check search starts from here
                      +    (fmh2 ? (*fmh2)[moved_piece][to_sq(move)] : VALUE_ZERO);
 
           // Increase reduction for cut nodes
-          if (!PvNode && cutNode)
+          if (cutNode)
               r += 2 * ONE_PLY;
 
           // Decrease reduction for moves that escape a capture. Filter out
-- 
2.39.2