From 9b30913996355f7d18eb18f1510620380921f1f2 Mon Sep 17 00:00:00 2001 From: Marco Costalba Date: Sat, 5 Jul 2014 08:15:57 +0200 Subject: [PATCH 1/1] Avoid 'double assigments' tricks Bitboard init code is already noteasy to follow, so don't make it even harder using 'smart' code. Also reindent a while loop in standard way. No functional change. --- src/bitboard.cpp | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/src/bitboard.cpp b/src/bitboard.cpp index d4b433fc..ff10b4ff 100644 --- a/src/bitboard.cpp +++ b/src/bitboard.cpp @@ -149,7 +149,10 @@ const std::string Bitboards::pretty(Bitboard b) { void Bitboards::init() { for (Square s = SQ_A1; s <= SQ_H8; ++s) - BSFTable[bsf_index(SquareBB[s] = 1ULL << s)] = s; + { + SquareBB[s] = 1ULL << s; + BSFTable[bsf_index(SquareBB[s])] = s; + } for (Bitboard b = 1; b < 256; ++b) MS1BTable[b] = more_than_one(b) ? MS1BTable[b - 1] : lsb(b); @@ -301,7 +304,8 @@ namespace { // Find a magic for square 's' picking up an (almost) random number // until we find the one that passes the verification test. do { - do magics[s] = rk.magic_rand(booster); + do + magics[s] = rk.magic_rand(booster); while (popcount((magics[s] * masks[s]) >> 56) < 6); std::memset(attacks[s], 0, size * sizeof(Bitboard)); -- 2.39.2