Skip to content

interpreters/python: Disable no-maybe-uninitialized warning#3583

Merged
acassis merged 1 commit into
apache:masterfrom
tmedicci:improvement/python_warning
Jul 3, 2026
Merged

interpreters/python: Disable no-maybe-uninitialized warning#3583
acassis merged 1 commit into
apache:masterfrom
tmedicci:improvement/python_warning

Conversation

@tmedicci

@tmedicci tmedicci commented Jul 2, 2026

Copy link
Copy Markdown
Contributor

Summary

  • interpreters/python: Disable no-maybe-uninitialized warning
    • Disables the no-maybe-uninitialized warning because it's wrongly evaluated after increasing the optimization level of
      the Python's library.

Impact

Impact on user: No.

Impact on build: Yes, enables building esp32s3-devkit:python with EXTRAFLAGS="-Wno-cpp -Werror"

Impact on hardware: No.

Impact on documentation: No.

Impact on security: No.

Impact on compatibility: No.

Testing

Building

make -j distclean && ./tools/configure.sh esp32s3-devkit:python && make EXTRAFLAGS="-Wno-cpp -Werror" -j$(nproc)

Running

Simply check build outputs.

Results

Check whether it's built properly.

This commit disables the `no-maybe-uninitialized` warning because
it's wrongly evaluated after increasing the optimization level of
the Python's library.

Signed-off-by: Tiago Medicci <tiago.medicci@espressif.com>

@cederom cederom left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you @tmedicci :-)

@acassis acassis merged commit 8e00b4d into apache:master Jul 3, 2026
41 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants