AndreasRichie / hsb_indicator

This project is meant to transfer all of the mold functionality to the Seeedstudio Sensecap Indicator. This includes all of the necessary UIs and backend logic.
0 stars 0 forks source link

Global Suppressed Warnings #13

Open schaufelbagger opened 1 month ago

schaufelbagger commented 1 month ago

Warnings are suppressed due to the -w flag given in the CMakeLists.txt in the ESP32 folder. While this masks unnecessary warnings and errors from libraries, it also suppresses warnings from the source code, which can lead to programming mistakes, i.e. missing a return statement in a non-void return function leading to a kernel panic.

schaufelbagger commented 1 month ago

The -w flag cannot be unset with another flag. Therefore it cannot be unset with the platform.ini. Also a build_src_unflags does not exist.

Also I tried the post script from here, which sets the -isystem flag to all non src files, but it did nothing.

Will check from the espressif docs: https://docs.espressif.com/projects/esp-idf/en/stable/esp32/api-guides/build-system.html

schaufelbagger commented 1 month ago

I removed the global warning flag and suppressed or fixed the warnings in the components. The enum-enum-conversion warning is still there as it is still argued in https://www.github.com/lvgl/lvgl/issues/5119 on what to do.