Move CXXFLAGS under COVERAGE handle

Author: yingyue2030699

.github/workflows/ci.yml

@@ -24,25 +24,21 @@ jobs:
       - name: Build tests with coverage
         run: |
           make clean
-          make test COVERAGE=1 CXX=g++ CXXFLAGS="-O0 -g --coverage" LDFLAGS="--coverage"
+          make test COVERAGE=1 CXX=g++
 
       - name: Run tests
         run: ./bin/test_main
 
       - name: Generate coverage (LCOV)
         run: |
-          # Clean any stale traces first
-          lcov --directory obj --directory bin --zerocounters
-          # Capture with matching gcov tool and stable options
           lcov --capture \
                --directory obj \
                --directory bin \
                --gcov-tool gcov-13 \
                --ignore-errors empty \
                --output-file coverage.info
-          # (optional) keep only your code
           lcov --remove coverage.info '/usr/*' '*/googletest/*' \
-              --output-file coverage.info
+               --output-file coverage.info
 
       - name: Upload to Codecov
         uses: codecov/codecov-action@v4

Makefile

@@ -54,6 +54,9 @@ ifeq (test,$(MAKECMDGOALS))
 	SDIR += tests
 endif
 
+# Set version to C++14
+CXXFLAGS = -std=c++14
+
 # Enable coverage for "coverage" and "test" targets
 # --- Coverage toggle (OFF by default)
 COVERAGE ?= 0
@@ -141,7 +144,6 @@ INTEL  = $(shell type icpc  >/dev/null 2>&1; echo $$?)
 GCC    = $(shell type g++   >/dev/null 2>&1; echo $$?)
 
 INCS    = $(shell gsl-config --cflags) -Iinclude -Iinclude/*
-CXXFLAGS += -std=c++14
 LIBS     = $(shell gsl-config --libs)
 
 # Add pthread to library if multithreading (embarrassingly parallel) is needed