Skip to content

Txiki v26#58

Open
lygstate wants to merge 3 commits into
lvgl:masterfrom
lygstate:txiki-v26
Open

Txiki v26#58
lygstate wants to merge 3 commits into
lvgl:masterfrom
lygstate:txiki-v26

Conversation

@lygstate
Copy link
Copy Markdown

@lygstate lygstate commented May 10, 2026


Summary by cubic

Upgrade txiki to v26.4.0 and update QuickJS calls to match the new API, restoring build compatibility with no behavior changes. Also fixes Simulator builds by disabling -Werror for unused-but-set-variable in CMake.

  • Dependencies

    • Bumped deps/txiki to v26.4.0.
  • Refactors

    • Switched JS array checks to new signature (JS_IsArray(value)) across components, style, and macros.
    • Updated boolean usage and property/symbol setup to match new API expectations.
    • Added an explicit return (JS_UNDEFINED) in the event stop-bubbling handler.

Written for commit 62ad778. Summary will update on new commits.

For #57

Copy link
Copy Markdown

@cubic-dev-ai cubic-dev-ai Bot left a comment

Choose a reason for hiding this comment

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

No issues found across 13 files

…riable]

[build] /mnt/c/work/lvgl/lv_binding_js/deps/txiki/deps/libwebsockets/lib/system/async-dns/async-dns.c:801:27: error: variable ‘c’ set but not used [-Werror=unused-but-set-variable]
[build]   801 |         lws_adns_cache_t *c;
[build]       |                           ^
[build] cc1: all warnings being treated as errors
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant