From 09b88f75c2bd4aab789eb09914e7e17050afa928 Mon Sep 17 00:00:00 2001
From: Stephen Seo <seo.disparate@gmail.com>
Date: Wed, 11 Sep 2024 13:11:55 +0900
Subject: [PATCH] Update Makefile

Some cleanup.
Use certain build flags only in "RELEASE" mode.
---
 Makefile | 14 ++++++++------
 1 file changed, 8 insertions(+), 6 deletions(-)

diff --git a/Makefile b/Makefile
index b8b0a29..b84edcb 100644
--- a/Makefile
+++ b/Makefile
@@ -5,9 +5,8 @@ COMMON_FLAGS = -Wall -Wextra -Wpedantic \
 DEBUG_FLAGS = -Og -g
 RELEASE_FLAGS = -O3 -DNDEBUG
 
-EXTRA_COMMON_FLAGS =
 ifndef MINIMAL_BUILD_FLAGS
-	EXTRA_COMMON_FLAGS = \
+	COMMON_FLAGS := ${COMMON_FLAGS} \
 		-Wformat -Wformat=2 -Wconversion -Wimplicit-fallthrough \
 		-Werror=format-security \
 		-U_FORTIFY_SOURCE -D_FORTIFY_SOURCE=3 \
@@ -18,16 +17,19 @@ ifndef MINIMAL_BUILD_FLAGS
 		-Wl,-z,relro -Wl,-z,now \
 		-Wl,--as-needed -Wl,--no-copy-dt-needed-entries \
 		-fPIE -pie \
-		-fno-delete-null-pointer-checks -fno-strict-overflow \
-		-fno-strict-aliasing -ftrivial-auto-var-init=zero \
 		-Werror=implicit -Werror=incompatible-pointer-types \
 		-Werror=int-conversion
+	ifdef RELEASE
+		COMMON_FLAGS := ${COMMON_FLAGS} \
+			-fno-delete-null-pointer-checks -fno-strict-overflow \
+			-fno-strict-aliasing -ftrivial-auto-var-init=zero
+	endif
 endif
 
 ifdef RELEASE
-	CFLAGS = ${COMMON_FLAGS} ${EXTRA_COMMON_FLAGS} ${RELEASE_FLAGS}
+	CFLAGS = ${COMMON_FLAGS} ${RELEASE_FLAGS}
 else
-	CFLAGS = ${COMMON_FLAGS} ${EXTRA_COMMON_FLAGS} ${DEBUG_FLAGS}
+	CFLAGS = ${COMMON_FLAGS} ${DEBUG_FLAGS}
 endif
 
 HEADERS = \