Skip to content

fix(settings_ui): drop esp_timer_set_arg use#128

Merged
baba-dev merged 1 commit into
mainfrom
codex/fix-implicit-declaration-error-in-settings_ui
Sep 19, 2025
Merged

fix(settings_ui): drop esp_timer_set_arg use#128
baba-dev merged 1 commit into
mainfrom
codex/fix-implicit-declaration-error-in-settings_ui

Conversation

@baba-dev
Copy link
Copy Markdown
Owner

Summary

  • replace the esp_timer_set_arg call with a static runtime pointer that works on the current IDF
  • clear the cached pointer when cancelling the dimmer to avoid stale references

Testing

  • ⚠️ idf.py build (command not available in the container)
  • ⚠️ scripts/idf.py build (missing /opt/esp/idf/tools/idf.py in the container)

https://chatgpt.com/codex/tasks/task_e_68cddb3ca3c08324b8d8de89c5c2cc8e

@baba-dev baba-dev merged commit 8825ed4 into main Sep 19, 2025
1 of 7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant