From: Fabian Beuke <mail@beuke.org>
Date: Sat, 19 Nov 2016 14:55:30 +0000 (+0100)
Subject: Reduce variable scope in swap_byte
X-Git-Url: https://git.sesse.net/?p=stockfish;a=commitdiff_plain;h=b5d10d17c225cccf0ed670b724a4cdcc4fc68018

Reduce variable scope in swap_byte

Added a specialization to remove the 'if' condition

No functional change.
---

diff --git a/src/syzygy/tbprobe.cpp b/src/syzygy/tbprobe.cpp
index b553ac9f..d13e9018 100644
--- a/src/syzygy/tbprobe.cpp
+++ b/src/syzygy/tbprobe.cpp
@@ -228,10 +228,10 @@ template<typename T, int Half = sizeof(T) / 2, int End = sizeof(T) - 1>
 inline void swap_byte(T& x)
 {
     char tmp, *c = (char*)&x;
-    if (Half) // Fix a MSVC 2015 warning
-        for (int i = 0; i < Half; ++i)
-            tmp = c[i], c[i] = c[End - i], c[End - i] = tmp;
+    for (int i = 0; i < Half; ++i)
+        tmp = c[i], c[i] = c[End - i], c[End - i] = tmp;
 }
+template<> inline void swap_byte<uint8_t, 0, 0>(uint8_t&) {}
 
 template<typename T, int LE> T number(void* addr)
 {