From: syzygy1 <3028851+syzygy1@users.noreply.github.com>
Date: Sat, 11 Jul 2020 14:25:34 +0000 (+0200)
Subject: Corrects a functional change in a cleanup patch.
X-Git-Url: https://git.sesse.net/?p=stockfish;a=commitdiff_plain;h=6c197c3964ca0c637ff1f646dc7e6653b1bb4b45

Corrects a functional change in a cleanup patch.

This corrects a functional change in
https://github.com/official-stockfish/Stockfish/commit/ddcbacd04d1c860e808202ce8c1206c8acdca627
changing evaluation of KPPvK. Bench remains unchanged at low depth

With this patch, 8/8/5k1p/8/7p/7K/8/8 b - - 1 11 is again correctly evaluated as a draw.

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

Bench: 4366686
---

diff --git a/src/endgame.cpp b/src/endgame.cpp
index 40f49dce..a8ceb648 100644
--- a/src/endgame.cpp
+++ b/src/endgame.cpp
@@ -589,8 +589,8 @@ ScaleFactor Endgame<KPsK>::operator()(const Position& pos) const {
   Bitboard strongPawns = pos.pieces(strongSide, PAWN);
 
   // If all pawns are ahead of the king on a single rook file, it's a draw.
-  if (!((strongPawns & ~FileABB) || (strongPawns & ~FileHBB)) &&
-      !(strongPawns & ~passed_pawn_span(weakSide, weakKing)))
+  if (   !(strongPawns & ~(FileABB | FileHBB))
+      && !(strongPawns & ~passed_pawn_span(weakSide, weakKing)))
       return SCALE_FACTOR_DRAW;
 
   return SCALE_FACTOR_NONE;