Declare test compat.h shims inline to avoid -Wunused-function (#24)

claude-wrapper.c sets but never unsets env after the LD_PRELOAD-overwrite
change, so the plain-`static` unsetenv shim was unused in that translation
unit and failed `-Werror` on Windows. `static inline` exempts header
helpers from -Wunused-function.

Co-authored-by: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

Author: gtbuchanan

test/compat.h

@@ -7,22 +7,25 @@
  * build to supply them; it is NEVER part of the shipped Termux build, so
  * production code stays free of host-portability cruft. On POSIX hosts the
  * block compiles to nothing and the real libc functions are used.
+ *
+ * The shims are `inline` so a translation unit that uses only one of them
+ * (e.g. claude-wrapper.c sets but never unsets) doesn't trip -Wunused-function.
  */
 #ifndef CLAUDE_WRAPPER_TEST_COMPAT_H
 #define CLAUDE_WRAPPER_TEST_COMPAT_H
 
 #if defined(_WIN32)
 #include <stdlib.h>
 
-static int setenv(const char *name, const char *value, int overwrite) {
+static inline int setenv(const char *name, const char *value, int overwrite) {
   if (!overwrite && getenv(name) != NULL) {
     return 0;
   }
   return _putenv_s(name, value);
 }
 
 /* _putenv_s(name, "") removes the variable on Windows (getenv → NULL). */
-static int unsetenv(const char *name) { return _putenv_s(name, ""); }
+static inline int unsetenv(const char *name) { return _putenv_s(name, ""); }
 #endif /* _WIN32 */
 
 #endif /* CLAUDE_WRAPPER_TEST_COMPAT_H */