Add Asan

Author: DoctorLai

.github/workflows/ci.yml

@@ -1,39 +1,52 @@
-name: C++ Build
+name: C++ CI
 
 on:
   push:
+    branches: [main]
   pull_request:
+    branches: [main]
 
 jobs:
-  build:
+  # Step 1: dynamically detect example folders
+  detect-examples:
     runs-on: ubuntu-latest
+    outputs:
+      folders: ${{ steps.set-folders.outputs.folders }}
+    steps:
+      - name: Checkout code
+        uses: actions/checkout@v4
+
+      - name: Detect example folders
+        id: set-folders
+        run: |
+          folders=$(find . -maxdepth 1 -type d -not -name '.' -exec test -f '{}/Makefile' \; -print | sed 's|^\./||' | tr '\n' ' ')
+          echo "Detected folders: $folders"
+          echo "::set-output name=folders::$folders"
 
+  # Step 2: build and run examples in parallel using a matrix
+  build-and-run:
+    needs: detect-examples
+    runs-on: ubuntu-latest
     strategy:
       matrix:
-        compiler: [g++]
+        folder: ${{ fromJSON('["' + needs.detect-examples.outputs.folders | replace(' ', '","') + '"]') }}
+        sanitizer: [address, thread, undefined]
 
     steps:
-      - name: Checkout repo
+      - name: Checkout code
         uses: actions/checkout@v4
 
       - name: Install build tools
         run: |
           sudo apt-get update
-          sudo apt-get install -y build-essential
+          sudo apt-get install -y build-essential g++ clang
 
-      - name: Build all examples
+      - name: Build example
         run: |
-          make
+          echo "Building ${{ matrix.folder }} with SANITIZE=${{ matrix.sanitizer }}"
+          make -C ${{ matrix.folder }} SANITIZE=${{ matrix.sanitizer }}
 
-      - name: Run examples
-        run: |
-          for dir in */ ; do
-            if [ -f "$dir/Makefile" ]; then
-              echo "Running $dir"
-              (cd "$dir" && make run || true)
-            fi
-          done
-
-      - name: Clean
+      - name: Run example
         run: |
-          make clean
+          echo "Running ${{ matrix.folder }} with SANITIZE=${{ matrix.sanitizer }}"
+          make -C ${{ matrix.folder }} SANITIZE=${{ matrix.sanitizer }} run

Makefile

@@ -1,13 +1,37 @@
+# ======================================
+# Top-level Makefile for cpp-coding-exercise
+# ======================================
+
+# Auto-detect all subdirectories that contain a Makefile
 SUBDIRS := $(dir $(wildcard */Makefile))
 
-.PHONY: all clean $(SUBDIRS)
+# Default sanitizer (can be overridden)
+SANITIZE ?= address
+
+.PHONY: all clean run $(SUBDIRS)
 
+# --------------------------
+# Build all examples
+# --------------------------
 all: $(SUBDIRS)
 
 $(SUBDIRS):
-	$(MAKE) -C $@
+	@echo "=== Building $@ with SANITIZE=$(SANITIZE) ==="
+	$(MAKE) -C $@ SANITIZE=$(SANITIZE)
+
+# --------------------------
+# Run all examples (optional)
+# --------------------------
+run:
+	@for dir in $(SUBDIRS); do \
+		echo "=== Running $$dir ==="; \
+		(cd $$dir && $(MAKE) SANITIZE=$(SANITIZE) run) || exit 1; \
+	done
 
+# --------------------------
+# Clean all examples
+# --------------------------
 clean:
-	for dir in $(SUBDIRS); do \
+	@for dir in $(SUBDIRS); do \
 		$(MAKE) -C $$dir clean; \
 	done

README.md

@@ -74,7 +74,7 @@ Runnable examples expose a `run` target:
 make run
 ```
 
-This keeps CI safe (build-only by default) while allowing local execution.
+`make run` on top level will run all built targets.
 
 ---
 
@@ -97,11 +97,11 @@ make clean
 
 ## Shared build configuration
 
-Common compiler settings live in `common.mk`:
+Common compiler settings live in [common.mk](./common.mk):
 
 ```make
 CXX      := g++
-CXXFLAGS := -std=c++20 -O2 -Wall -Wextra
+CXXFLAGS := -std=c++20 -Wall -Wextra
 ```
 
 Individual examples may extend this, e.g.:
@@ -112,6 +112,25 @@ CXXFLAGS += -pthread
 
 ---
 
+## Address Sanitizer Check
+The Address Sanitizer is enabled by default to ensure there is no memory leaks or other memory problems.
+
+```make
+# Builds with AddressSanitizer automatically
+make
+
+# ThreadSanitizer
+make SANITIZE=thread
+
+# UndefinedBehaviorSanitizer
+make SANITIZE=undefined
+
+# No sanitizers
+make SANITIZE=
+```
+
+---
+
 ## Continuous Integration
 
 GitHub Actions automatically builds all examples on:

common.mk

@@ -1,2 +1,48 @@
+# ======================================
+# common.mk - shared compiler flags
+# ======================================
+
 CXX      := g++
-CXXFLAGS := -std=c++20 -O2 -Wall -Wextra
+OPTFLAGS ?= -O1 -g           # Safe for ASAN/TSAN (default)
+
+# --------------------------
+# Sanitizer selection
+# --------------------------
+# Default: AddressSanitizer
+SANITIZE ?= address
+
+ifeq ($(SANITIZE),address)
+    SAN_FLAGS := -fsanitize=address
+else ifeq ($(SANITIZE),thread)
+    SAN_FLAGS := -fsanitize=thread
+else ifeq ($(SANITIZE),undefined)
+    SAN_FLAGS := -fsanitize=undefined
+else ifeq ($(SANITIZE),)
+    SAN_FLAGS :=
+else
+    $(error Unknown SANITIZE=$(SANITIZE))
+endif
+
+# --------------------------
+# Compiler / Linker flags
+# --------------------------
+CXXFLAGS := -std=c++20 -Wall -Wextra $(OPTFLAGS) $(SAN_FLAGS)
+LDFLAGS  := $(SAN_FLAGS)
+
+# --------------------------
+# Usage notes
+# --------------------------
+# Local build (default): ASAN enabled
+#   $ make
+#
+# Override for ThreadSanitizer:
+#   $ make SANITIZE=thread
+#
+# Override for UBSAN:
+#   $ make SANITIZE=undefined
+#
+# Disable sanitizers (rare):
+#   $ make SANITIZE=
+#
+# LDFLAGS are automatically applied in example Makefiles:
+#   $(CXX) $(CXXFLAGS) -o $(TARGET) $(OBJS) $(LDFLAGS)