Copied Nacelle project config updates over

Author: BobSaidHi

platformio.ini

@@ -8,6 +8,9 @@
 ; Please visit documentation for the other options and examples
 ; https://docs.platformio.org/page/projectconf.html
 
+[platformio]
+default_envs = Prod_Load
+
 ; Common - https://docs.platformio.org/en/latest/projectconf/sections/env/index.html#common-env
 [env]
 ; See also: https://esp32s3.com/getting-started.html
@@ -23,11 +26,24 @@ lib_deps =
     ayushsharma82/ElegantOTA @ ^3.1.7
     etlcpp/Embedded Template Library @ ^20.44.1
     lennarthennigs/ESP Telnet @ ^2.2.3
+; https://docs.platformio.org/en/latest/projectconf/sections/env/options/build/build_flags.html
 ; https://techoverflow.net/2021/12/14/how-to-print-all-preprocessor-flags-in-platformio/
-; build_flags = -O3 -std=gnu++23 -Wall -E -dM -DCORE_DEBUG_LEVEL=ARDUHAL_LOG_LEVEL_DEBUG -D ELEGANTOTA_USE_ASYNC_WEBSERVER=1  -D CONFIG_ASYNC_TCP_STACK_SIZE=4096
-; build_flags = -O3 -std=gnu++23 -latomic -DCORE_DEBUG_LEVEL=ARDUHAL_LOG_LEVEL_DEBUG -D ELEGANTOTA_USE_ASYNC_WEBSERVER=1  -D CONFIG_ASYNC_TCP_STACK_SIZE=4096
-build_flags = -O3 -std=gnu++23 -Wall -DCORE_DEBUG_LEVEL=ARDUHAL_LOG_LEVEL_DEBUG -D ELEGANTOTA_USE_ASYNC_WEBSERVER=1  -D CONFIG_ASYNC_TCP_STACK_SIZE=4096
-build_unflags = -Os -std=gnu++11 -std=gnu++14 -std=gnu++17 -std=gnu++20 -std=gnu++2b -std=gnu++2a
+; See also: https://interrupt.memfault.com/blog/best-and-worst-gcc-clang-compiler-flags
+; Note: Not using  -Wpedantic, -Wundef, -Wconversion, -Wsign-conversion, & -Wpadded as it causes to warnings in libraries.   -Wshadow is also a bit too much. -Wstack-usage=<stack_limit> not used because it varies.
+; build_flags =
+;    -O3 -ffunction-sections -fdata-sections -Wl,--gc-sections
+;    -std=gnu++23
+;  	 -Wall -Wextra -Wdouble-promotion -Wformat=2 -Wformat-overflow=2 -Wformat-truncation=2 -fno-common -fstack-usage -Woverloaded-virtual
+;    -E -dM -DCORE_DEBUG_LEVEL=ARDUHAL_LOG_LEVEL_DEBUG -D ELEGANTOTA_USE_ASYNC_WEBSERVER=1  -D CONFIG_ASYNC_TCP_STACK_SIZE=4096
+; build_flags = -O3 -ffunction-sections -fdata-sections -Wl,--gc-sections -std=gnu++23 -latomic -D ELEGANTOTA_USE_ASYNC_WEBSERVER=1 -D CONFIG_ASYNC_TCP_STACK_SIZE=4096
+build_flags =
+    -O3 -ffunction-sections -fdata-sections -Wl,--gc-sections
+    -std=gnu++23
+    -Wall -Wextra -Wdouble-promotion -Wformat=2 -Wformat-overflow=2 -Wformat-truncation=2 -fno-common -fstack-usage -Woverloaded-virtual
+    -DCORE_DEBUG_LEVEL=ARDUHAL_LOG_LEVEL_DEBUG -D ELEGANTOTA_USE_ASYNC_WEBSERVER=1  -D CONFIG_ASYNC_TCP_STACK_SIZE=4096
+build_unflags =
+    -Os
+    -std=gnu++11 -std=gnu++14 -std=gnu++17 -std=gnu++20 -std=gnu++2b -std=gnu++2a
 ; https://docs.platformio.org/en/latest/advanced/static-code-analysis/index.html
 ; MISRA Rules: https://github.com/speeduino/speeduino/tree/master/misra
 ; PVS-Studio License Comments: https://pvs-studio.com/en/blog/posts/0614/
@@ -46,9 +62,9 @@ check_flags =
 board = um_pros3
 ; Extra options already in boards file
 build_flags =
-	-O3
-	-std=gnu++23
-	-Wall
+    -O3 -ffunction-sections -fdata-sections -Wl,--gc-sections
+    -std=gnu++23
+    -Wall
     -DCORE_DEBUG_LEVEL=ARDUHAL_LOG_LEVEL_DEBUG
     -I"${platformio.packages_dir}/framework-arduinoespressif32-libs/esp32s3/include/esp_driver_tsens/include/driver"
     ; 	-D ARDUINO_USB_MODE=1 ; CDC/JTAG USB mode
@@ -72,17 +88,17 @@ build_flags =
 ; board = esp32-c3-devkitm-1
 ; Extra options already in boards file
 ; build_flags =
-; 	-O3
+; 	-O3 -ffunction-sections -fdata-sections -Wl,--gc-sections
 ; 	-std=gnu++23
-; 	-Wall
-;     -DCORE_DEBUG_LEVEL=ARDUHAL_LOG_LEVEL_DEBUG
-;     -I"${platformio.packages_dir}/framework-arduinoespressif32-libs/esp32/include/esp_driver_tsens/include/driver"
-;     ; 	-D ARDUINO_USB_MODE=1 ; CDC/JTAG USB mode
-;     ; 	-D ARDUINO_USB_CDC_ON_BOOT=1 ; Needed for USB to enumerate when user code starts
-;     -D ELEGANTOTA_USE_ASYNC_WEBSERVER=1
-;     ; -D CONFIG_ASYNC_TCP_MAX_ACK_TIME=5000   ; (keep default)
-;     ; -D CONFIG_ASYNC_TCP_PRIORITY=10         ; (keep default)
-;     ; -D CONFIG_ASYNC_TCP_QUEUE_SIZE=64       ; (keep default)
-;     -D CONFIG_ASYNC_TCP_RUNNING_CORE=1      ; force async_tcp task to be on same core as the app (default is core 0)
-;     -D CONFIG_ASYNC_TCP_STACK_SIZE=4096     ; reduce the stack size (default is 16K)
+; 	-Wall -Wextra -Wdouble-promotion -Wformat=2 -Wformat-overflow=2 -Wformat-truncation=2 -fno-common -fstack-usage -Woverloaded-virtual
+;   -DCORE_DEBUG_LEVEL=ARDUHAL_LOG_LEVEL_DEBUG
+;   -I"${platformio.packages_dir}/framework-arduinoespressif32-libs/esp32/include/esp_driver_tsens/include/driver"
+;   ; 	-D ARDUINO_USB_MODE=1 ; CDC/JTAG USB mode
+;   ; 	-D ARDUINO_USB_CDC_ON_BOOT=1 ; Needed for USB to enumerate when user code starts
+;   -D ELEGANTOTA_USE_ASYNC_WEBSERVER=1
+;   ; -D CONFIG_ASYNC_TCP_MAX_ACK_TIME=5000   ; (keep default)
+;   ; -D CONFIG_ASYNC_TCP_PRIORITY=10         ; (keep default)
+;   ; -D CONFIG_ASYNC_TCP_QUEUE_SIZE=64       ; (keep default)
+;   -D CONFIG_ASYNC_TCP_RUNNING_CORE=1      ; force async_tcp task to be on same core as the app (default is core 0)
+;   -D CONFIG_ASYNC_TCP_STACK_SIZE=4096     ; reduce the stack size (default is 16K)