From d6da63bf8c71bd8797521655848e372dadd8b37e Mon Sep 17 00:00:00 2001
From: sawinglogz <3787776-sawinglogz@users.noreply.gitlab.com>
Date: Sun, 4 Aug 2019 20:21:02 -0500
Subject: [PATCH] Always treat warnings as errors, even (especially!) in
 release.

Also rename the debug GUI build to "memdebug" to avoid colliding with
Qt's built-in debug build.
---
 oscar/oscar.pro | 9 +++++++--
 1 file changed, 7 insertions(+), 2 deletions(-)

diff --git a/oscar/oscar.pro b/oscar/oscar.pro
index eb8e111e..fc103de8 100644
--- a/oscar/oscar.pro
+++ b/oscar/oscar.pro
@@ -450,8 +450,13 @@ DISTFILES += help/default.css \
     help/index.qhcp
 }
 
-# Create a debug GUI build by adding "CONFIG+=debug" to your qmake command
-debug {
+
+# Always treat warnings as errors, even (especially!) in release
+QMAKE_CFLAGS += -Werror
+QMAKE_CXXFLAGS += -Werror
+
+# Create a debug GUI build by adding "CONFIG+=memdebug" to your qmake command
+memdebug {
     !win32 {  # add memory checking on Linux and macOS debug builds
         QMAKE_CFLAGS += -g -Werror -fsanitize=address -fno-omit-frame-pointer -fno-common -fsanitize-address-use-after-scope
         QMAKE_CXXFLAGS += -g -Werror -fsanitize=address -fno-omit-frame-pointer -fno-common -fsanitize-address-use-after-scope