Skip to content

backend/wayland: Use calloop executor to spawn future#293

Draft
ids1024 wants to merge 1 commit intomasterfrom
calloop
Draft

backend/wayland: Use calloop executor to spawn future#293
ids1024 wants to merge 1 commit intomasterfrom
calloop

Conversation

@ids1024
Copy link
Member

@ids1024 ids1024 commented Mar 12, 2026

This should be fixed now that we're using calloop 0.14.4.

Spawning a thread for futures is excessive if we only use it to wait for buffer release.

This seems to be good, but testing this I'm looking into some inconsistent issues that seem to happen with or without this (though it's hard to be sure if there's a change).

  • I have disclosed use of any AI generated code in my commit messages.
    • If you are using an LLM, and do not fully understand the changes it is making to the code base, do not create a PR.
    • In our experience, AI generated code often results in overly complex code that lacks enough context for a proper fix or feature inclusion. This results in considerably longer code reviews. Due to this, AI authored or partially authored PRs may be closed without comment.
  • I understand these changes in full and will be able to respond to review comments.
  • My change is accurately described in the commit message.
  • My contribution is tested and working as described.
  • I have read the Developer Certificate of Origin and certify my contribution under its conditions.

This should be fixed now that we're using `calloop` 0.14.4.

Spawning a thread for futures is excessive if we only use it to wait for
buffer release.
@ids1024 ids1024 changed the title backend/wayland: Use calloop executor to spwan future backend/wayland: Use calloop executor to spawn future Mar 13, 2026
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