Added compiler.warning_flags to platform.txt; fixing #6118

[PilnyTomas]

Summary

Added compiler.warning_flags to all chips in platform.txt to reflect users setting of warning level output during compilation (set up in Arduino IDE preferences)

Impact

When a warning is set to none the compilation will no longer display warnings

Related links

Solves issue #6118

[github-actions]

Unit Test Results

0 files  0 suites   0s ⏱️
0 tests 0 ✔️ 0 💤 0 ❌

Results for commit d68c66f.

♻️ This comment has been updated with latest results.

[mrengineer7777]

Why does this PR modify "cores/esp32/esp32-hal-i2c-slave.c" and "libraries/WiFiClientSecure/src/esp_crt_bundle.c"?

[PilnyTomas]

Why does this PR modify "cores/esp32/esp32-hal-i2c-slave.c" and "libraries/WiFiClientSecure/src/esp_crt_bundle.c"?

With the new compiler settings, the previously hidden warnings are now treated as errors and the CI is failing. Following commits are simply satisfying the compiler/CI. The changes are mostly removing unused variables and such.

[mrengineer7777]

With the new compiler settings, the previously hidden warnings are now treated as errors and the CI is failing. Following commits are simply satisfying the compiler/CI. The changes are mostly removing unused variables and such.

Makes sense. I glanced over the code and don't see any obvious issues.

[VojtechBartoska]

any updates on this @PilnyTomas?

[me-no-dev]

Thanks @PilnyTomas for your hard work :)