From 25ccc9f2d51a048d664bbceaedf2877660c75a79 Mon Sep 17 00:00:00 2001
From: Enrico Loparco <eloparco@amazon.com>
Date: Wed, 17 Jan 2024 01:06:02 +0100
Subject: [PATCH] Avoid unused thread_id warning and recompile multi-module
 sample (#3033)

- Unused variable warning (when building in release mode):
```bash
wasm-micro-runtime/samples/wasi-threads/wasm-apps/no_pthread.c:44:9:
warning: variable 'thread_id' set but not used [-Wunused-but-set-variable]
    int thread_id;
```
- When using the multi-module example and changing the files in `wasm-apps`,
  they don't get recompiled when the sample is rebuilt. Adding `BUILD_ALWAYS`
  as it's done in other samples.
---
 samples/multi-module/CMakeLists.txt                | 1 +
 samples/wasi-threads/wasm-apps/wasi_thread_start.h | 1 +
 2 files changed, 2 insertions(+)

diff --git a/samples/multi-module/CMakeLists.txt b/samples/multi-module/CMakeLists.txt
index ee6c98df..17f4e1bc 100644
--- a/samples/multi-module/CMakeLists.txt
+++ b/samples/multi-module/CMakeLists.txt
@@ -132,6 +132,7 @@ endif()
 # .c -> .wasm
 ExternalProject_Add(WASM_MODULE
   SOURCE_DIR         ${CMAKE_CURRENT_SOURCE_DIR}/wasm-apps
+  BUILD_ALWAYS TRUE
   UPDATE_COMMAND     ""
   PATCH_COMMAND      ""
   CONFIGURE_COMMAND  ${CMAKE_COMMAND}
diff --git a/samples/wasi-threads/wasm-apps/wasi_thread_start.h b/samples/wasi-threads/wasm-apps/wasi_thread_start.h
index 2427fd2b..5eae2938 100644
--- a/samples/wasi-threads/wasm-apps/wasi_thread_start.h
+++ b/samples/wasi-threads/wasm-apps/wasi_thread_start.h
@@ -9,6 +9,7 @@
 
 /* See https://github.com/WebAssembly/wasi-threads#design-choice-thread-ids */
 #define ASSERT_VALID_TID(TID) \
+    (void)TID;                \
     assert(TID >= 1 && TID <= 0x1FFFFFFF && "Invalid thread ID")
 
 typedef struct {