From a9d09b9658ecc9d2c55c9b49ea9185d0a9392223 Mon Sep 17 00:00:00 2001
From: Frank <91616163+softhack007@users.noreply.github.com>
Date: Fri, 14 Oct 2022 14:33:37 +0200
Subject: [PATCH] enable more compiler warnings

---
 platformio.ini | 8 ++++++++
 1 file changed, 8 insertions(+)

diff --git a/platformio.ini b/platformio.ini
index 414613ec..8fdd1780 100644
--- a/platformio.ini
+++ b/platformio.ini
@@ -533,6 +533,8 @@ build_unflags = ${common.build_unflags}
 
 build_flags = ${common.build_flags_esp32}
   -D WLED_RELEASE_NAME=ESP32 #-D WLED_DISABLE_BLYNK #-D WLED_DISABLE_BROWNOUT_DET
+  -Wall -Wformat -Woverflow -Wuninitialized -Winit-self -Warray-bounds ; enables more warnings
+  -Wno-attributes -Wno-unused-variable -Wno-unused-function ; disables some stupid warnings
   -D ABL_MILLIAMPS_DEFAULT=1500 ; 850 not enough for 1024 leds
   -D WLED_USE_MY_CONFIG
   -D USERMOD_AUDIOREACTIVE
@@ -556,6 +558,8 @@ platform_packages = ${esp32.platform_packages}
 build_unflags = ${common.build_unflags}
 build_flags = ${common.build_flags_esp32}
   -D WLED_RELEASE_NAME=ESP32 ; This will be included in the firmware.bin filename
+  ; -Wall -Wformat -Woverflow -Wuninitialized -Winit-self -Warray-bounds ; enables more warnings
+  ; -Wno-attributes -Wno-unused-variable -Wno-unused-function ; disables some stupid warnings
   -D WLED_DISABLE_BLYNK ; BLYNK is only provided for backwards compatibility (no new users accepted)
   ;-D WLED_DISABLE_BROWNOUT_DET ; enable if you get "brownout detected" errors at startup 
   -D ABL_MILLIAMPS_DEFAULT=1500 ; 850 not enough for 1024 leds
@@ -623,6 +627,8 @@ build_unflags = ${common.build_unflags}
 build_flags =  ${common.build_flags} ${esp32.build_flagsV4} 
   -D WLED_RELEASE_NAME=ESP32 -D WLED_WATCHDOG_TIMEOUT=0 #-D WLED_DISABLE_BLYNK #-D WLED_DISABLE_BROWNOUT_DET
   -D ARDUINO_USB_CDC_ON_BOOT=0 ; needed for arduino-esp32 >=2.0.4; avoids errors on startup
+  -Wall -Wformat -Woverflow -Wuninitialized -Winit-self -Warray-bounds ; enables more warnings
+  -Wno-attributes -Wno-misleading-indentation -Wno-unused-variable -Wno-unused-function -Wno-deprecated-declarations ; disables some stupid warnings
   -D WLED_USE_PSRAM 
   -D ABL_MILLIAMPS_DEFAULT=1500 ; 850 not enough for 1024 leds
   -D WLED_USE_MY_CONFIG
@@ -650,6 +656,8 @@ upload_speed = 460800 ; or 921600
 build_unflags = ${common.build_unflags}
 build_flags =  ${common.build_flags} ${esp32s3.build_flags} -D WLED_RELEASE_NAME=ESP32S3 #-D WLED_DISABLE_BLYNK #-D WLED_DISABLE_BROWNOUT_DET
   -D ARDUINO_USB_MODE=1 -D ARDUINO_USB_CDC_ON_BOOT=0 -D ARDUINO_USB_MSC_ON_BOOT=0 -D ARDUINO_USB_DFU_ON_BOOT=0
+  -Wall -Wformat -Woverflow -Wuninitialized -Winit-self -Warray-bounds ; enables more warnings
+  -Wno-attributes -Wno-misleading-indentation -Wno-unused-variable -Wno-unused-function -Wno-deprecated-declarations ; disables some stupid warnings
   -D WLED_WATCHDOG_TIMEOUT=0 
   -D ABL_MILLIAMPS_DEFAULT=1500 ; 850 not enough for 1024 leds
   -D LEDPIN=4