Fix a warning with MSVC in 'analyze' mode
authorMarco Costalba <mcostalba@gmail.com>
Sat, 14 Jun 2014 10:46:58 +0000 (12:46 +0200)
committerMarco Costalba <mcostalba@gmail.com>
Sat, 14 Jun 2014 10:46:58 +0000 (12:46 +0200)
Here MSVC is worried that

StepAttacksBB[PAWN][psq]

could overflow, so change psq initialization
to clarify psq is always less than 64.

No functional change.

src/bitbase.cpp

index 6094f50fbe9da0b9bca1bc3e47927bc35f43f977..0f274c8aeb82a7d1d7ab0ddf3cfc6833d6f7b570 100644 (file)
@@ -107,11 +107,11 @@ namespace {
 
   KPKPosition::KPKPosition(unsigned idx) {
 
-    wksq = Square((idx >>  0) & 0x3F);
-    bksq = Square((idx >>  6) & 0x3F);
-    us   = Color ((idx >> 12) & 0x01);
-    psq  = make_square(File((idx >> 13) & 0x03), Rank(RANK_7 - (idx >> 15)));
-    result  = UNKNOWN;
+    wksq   = Square((idx >>  0) & 0x3F);
+    bksq   = Square((idx >>  6) & 0x3F);
+    us     = Color ((idx >> 12) & 0x01);
+    psq    = make_square(File((idx >> 13) & 0x3), RANK_7 - Rank((idx >> 15) & 0x7));
+    result = UNKNOWN;
 
     // Check if two pieces are on the same square or if a king can be captured
     if (   square_distance(wksq, bksq) <= 1