dangernoodle-io / breadboard / 31233803047

97%

push

github

fix: add missing <stdio.h> for snprintf callers (#1250)

bb_data_http_common.c and test_bb_log.c call snprintf() without
directly including <stdio.h>. gcc 14+ defaults to C23, where an
implicit function declaration is a hard compile error, not a
warning; a genuinely-GNU local toolchain (e.g. Homebrew gcc-16,
which this repo's coverage_toolchain.sh accepts) is one bump away
from failing here. Today both call sites happen to compile only
because they transitively inherit stdio.h (bb_log.h's host
fprintf/putchar branch, Unity's default UNITY_OUTPUT_CHAR) -- an
incidental, fragile path, not a guarantee.

- adds only the missing include to each file, no other changes


Claude-Session: https://claude.ai/code/session_0119nuE2NZME9y2aYEVJTPB5

Co-authored-by: Claude Opus 5 <noreply@anthropic.com>

8544 of 9084 branches covered (94.06%)

Branch coverage included in aggregate %.

14274 of 14429 relevant lines covered (98.93%)

2603.86 hits per line