From fcecc5212e42794cba218fc1ffd39cd9da818655 Mon Sep 17 00:00:00 2001 From: Marco Costalba Date: Sun, 15 Mar 2009 18:06:40 +0100 Subject: [PATCH 1/1] Fix an overflow bug in pawns stormValue These fields are defined as int8_t but values bigger then 127 are stored there so that we silently overflow. Fix bringing up all the fields to a sane int type. This will increase memory usage, but apart from being safe, it is not clear if code is slower or faster. Test is needed. Signed-off-by: Marco Costalba --- src/pawns.h | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/src/pawns.h b/src/pawns.h index 7d3e9252..673bcfa5 100644 --- a/src/pawns.h +++ b/src/pawns.h @@ -58,9 +58,9 @@ private: Key key; Bitboard passedPawns; - int16_t mgValue, egValue; - int8_t ksStormValue[2], qsStormValue[2]; - uint8_t halfOpenFiles[2]; + int mgValue, egValue; + int ksStormValue[2], qsStormValue[2]; + int halfOpenFiles[2]; }; -- 2.39.2