fix(mqtt_cxx): Fix clang-tidy warnings with C++ exceptions #809

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Merged

euripedesrocha merged 1 commit into espressif:master from david-cermak:fix/mqtt_cxx_clang_tidy_warns

May 6, 2025

Collaborator

david-cermak commented May 2, 2025

most of the warnings (like https://github.com/espressif/esp-protocols/security/code-scanning/11) are showing as exceptions are disabled by default
mqtt-cxx client needs exceptions
Closes Fix code scanning alert - expected '(' for function-style cast or type construction [clang-diagnostic-error] (IDFGH-12824) #572


          fix(mqtt_cxx): Fix clang-tidy warnings with C++ exceptions

40cce5e

david-cermak requested a review from euripedesrocha

May 2, 2025 08:22

david-cermak self-assigned this

david-cermak added the mqtt label

euripedesrocha approved these changes

View reviewed changes

euripedesrocha merged commit b8cdd37 into espressif:master

84 checks passed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

mqtt