Small simplification in gives_check

Use switch statement also for normal case.

No speed regression.

No functional change.

diff --git a/src/position.cpp b/src/position.cpp
index 3fc568eebf93a0e674d748ceee4095c8b92b7551..1cb64934e95c2ace279d78a5d31f9a5e5c9a4f77 100644 (file)
--- a/src/position.cpp
+++ b/src/position.cpp
@@ -596,12 +596,11 @@ bool Position::gives_check(Move m, const CheckInfo& ci) const {
       && !aligned(from, to, ci.ksq))
       return true;
 
       && !aligned(from, to, ci.ksq))
       return true;
 

-  // Can we skip the ugly special cases?
-  if (type_of(m) == NORMAL)
-      return false;
-

   switch (type_of(m))
   {
   switch (type_of(m))
   {

+  case NORMAL:
+      return false;
+

   case PROMOTION:
       return attacks_bb(Piece(promotion_type(m)), to, pieces() ^ from) & ci.ksq;
 
   case PROMOTION:
       return attacks_bb(Piece(promotion_type(m)), to, pieces() ^ from) & ci.ksq;