Skip to content
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.

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

Move main loop handling code to library_eventloop.js. NFC #22615

Merged
merged 1 commit into from
Sep 25, 2024

Conversation

sbc100
Copy link
Collaborator

@sbc100 sbc100 commented Sep 24, 2024

This means that programs that use the main event loops don't need to depend on the Browser object.

We already supported and test the main loops handling code on node, but programs that only use the event loop without any other Browser functions no longer need to pull in the whole Browser object.

@sbc100 sbc100 changed the title Move main loops handling code to library_eventloop.js. NFC Move main loop handling code to library_eventloop.js. NFC Sep 24, 2024
@sbc100 sbc100 requested a review from kripken September 24, 2024 19:50
@sbc100 sbc100 force-pushed the eventloop branch 4 times, most recently from 5b521e6 to d073cec Compare September 24, 2024 21:38
@sbc100 sbc100 enabled auto-merge (squash) September 24, 2024 22:14
This means that programs that use the main event loops don't need to
depend on the `Browser` object.

We already supported and test the main loops handling code on node, but
programs that only use the event loop without any other `Browser`
functions no longer need to pull in the whole `Browser` object.
@sbc100 sbc100 merged commit 52cc139 into emscripten-core:main Sep 25, 2024
28 checks passed
@sbc100 sbc100 deleted the eventloop branch September 25, 2024 03:10
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.

2 participants