From 6d76699e44f2ed0fb2368ccf0a19ae692a2d4697 Mon Sep 17 00:00:00 2001 From: Scott Gasch Date: Thu, 3 Sep 2026 17:01:58 -0700 Subject: Fix all build warnings across release/DEBUG/TEST profiles Clean gmake GENETIC=1 PERF_COUNTERS=1 MP=1 SIXTYFOUR=1 build had 100 warnings; DEBUG=1 and TEST=1 builds had more once actually exercised. - OFFSET_OF/CONTAINING_STRUCT (chess.h) and PTR_TO_ALLOC_HASH (unix.c) truncated pointers through 32-bit ULONG before use in offset/hash arithmetic on this 64-bit build -- routed through size_t instead. - Diagnostic int<->void* round-trips (command.c, root.c, split.c, sig.c, data.c, unix.c, util.c) widened/narrowed via size_t to avoid implicit truncation. - ABS_DIFF on unsigned COOR now casts to int before abs(). - Dropped -fexpensive-optimizations (GCC-only, clang silently ignores it) from GNUmakefile. - Removed genuinely dead variables (book.c, gamelist.c, split.c, testgenerate.c, testhash.c). - Guarded DEBUG/PERF_COUNTERS/_X86_-only variables and the _CMEvidenceBucket helper under the #ifdef that actually reads them, since ASSERT/EVAL_TERM/KEEP_TRACK_OF_FIRST_MOVE_FHs compile away outside those builds. - Added missing prototypes for SlidePawn, SlidePawnWithoutSigs, SlidePieceWithoutSigs (move.c), previously undeclared in chess.h. - Removed dead _SystemIsRoot (unix.c). Verified via precommit_check.sh: TEST=1 self-test suite passes, DEBUG=1 smoke test (10 random ECM positions, sd 4) passes with no crashes/assertions, release build restored -- all three profiles now build with zero warnings. Co-Authored-By: Claude Sonnet 5 Claude-Session: https://claude.ai/code/session_014Cmv11sJZqVfanrPh6UnWE --- src/eval.c | 9 +++++---- 1 file changed, 5 insertions(+), 4 deletions(-) (limited to 'src/eval.c') diff --git a/src/eval.c b/src/eval.c index 2beccbf..3e856f1 100755 --- a/src/eval.c +++ b/src/eval.c @@ -2330,10 +2330,10 @@ Return value: ASSERT(GET_COLOR(p) == xSide); p = 1 << PIECE_TYPE(p); - uCounter += ((i == 0) | (i == -2) | - ((CHECK_VECTOR_WITH_INDEX(i, xSide) & p) != 0) | - ((CHECK_VECTOR_WITH_INDEX(i + 1, xSide) & p) != 0) | - ((CHECK_VECTOR_WITH_INDEX(i + 2, xSide) & p) != 0)); + uCounter += (int)(i == 0) | (int)(i == -2) | + (int)((CHECK_VECTOR_WITH_INDEX(i, xSide) & p) != 0) | + (int)((CHECK_VECTOR_WITH_INDEX(i + 1, xSide) & p) != 0) | + (int)((CHECK_VECTOR_WITH_INDEX(i + 2, xSide) & p) != 0); } ASSERT(uCounter < 15); @@ -4610,6 +4610,7 @@ Return value: if (_WhoControlsSquareFast(pos, c) == FLIP(uColor)) { p = pos->rgSquare[c].pPiece; + (void)p; #ifdef DEBUG ASSERT(p); ASSERT(!IS_PAWN(p)); -- cgit v1.3