Remove threatenedByPawn from rook threat

Can be simplified away. Passed STC: https://tests.stockfishchess.org/tests/view/65a3fa4179aa8af82b96face LLR: 2.92 (-2.94,2.94) <-1.75,0.25> Total: 30592 W: 7903 L: 7674 D: 15015 Ptnml(0-2): 96, 3590, 7711, 3787, 112 Passed LTC: https://tests.stockfishchess.org/tests/view/65a42b9a79aa8af82b96fe88 LLR: 2.95 (-2.94,2.94) <-1.75,0.25> Total: 73656 W: 18382 L: 18212 D: 37062 Ptnml(0-2): 47, 8287, 19981, 8475, 38 closes #4993 Bench: 1430061
official-stockfish · Jan 17, 2024 · 9a9702d · 9a9702d
1 parent 0fbad56
commit 9a9702d
Showing 1 changed file with 7 additions and 9 deletions.
diff --git a/src/movepick.cpp b/src/movepick.cpp
@@ -198,15 +198,13 @@ void MovePicker::score() {
                                                : 0;
 
             // malus for putting piece en prise
-            m.value -=
-              !(threatenedPieces & from)
-                ? (pt == QUEEN
-                     ? bool(to & threatenedByRook) * 50000 + bool(to & threatenedByMinor) * 10000
-                   : pt == ROOK
-                     ? bool(to & threatenedByMinor) * 25000 + bool(to & threatenedByPawn) * 10000
-                   : pt != PAWN ? bool(to & threatenedByPawn) * 15000
-                                : 0)
-                : 0;
+            m.value -= !(threatenedPieces & from)
+                       ? (pt == QUEEN ? bool(to & threatenedByRook) * 50000
+                                          + bool(to & threatenedByMinor) * 10000
+                          : pt == ROOK ? bool(to & threatenedByMinor) * 25000
+                          : pt != PAWN ? bool(to & threatenedByPawn) * 15000
+                                       : 0)
+                       : 0;
         }
 
         else  // Type == EVASIONS