Skip to content

[file_packager] Convert preload caching functions to async. NFC #24885

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

Merged
merged 2 commits into from
Aug 8, 2025

Conversation

sbc100
Copy link
Collaborator

@sbc100 sbc100 commented Aug 8, 2025

The 3 functions related to IDB caching of preload data are now marked as async. This means that they return promises instead of using callbacks.

Followup to #24882 and #24883

@sbc100
Copy link
Collaborator Author

sbc100 commented Aug 8, 2025

Part 3. Also best viewed with ignore whitespace.

@sbc100 sbc100 force-pushed the file_packager_async_part3 branch 2 times, most recently from 31887e3 to ae82d77 Compare August 8, 2025 15:56
@sbc100 sbc100 changed the title [file_packager] Convert checkCachedPackage to async/await. NFC [file_packager] Convert preload caching function to async. NFC Aug 8, 2025
@sbc100 sbc100 changed the title [file_packager] Convert preload caching function to async. NFC [file_packager] Convert preload caching functions to async. NFC Aug 8, 2025
@sbc100 sbc100 force-pushed the file_packager_async_part3 branch 2 times, most recently from 89deaf7 to 5f9ae76 Compare August 8, 2025 16:24
@sbc100 sbc100 force-pushed the file_packager_async_part3 branch from 5f9ae76 to 02b795e Compare August 8, 2025 16:40
sbc100 added a commit that referenced this pull request Aug 8, 2025
@sbc100 sbc100 merged commit 592d12d into emscripten-core:main Aug 8, 2025
30 checks passed
@sbc100 sbc100 deleted the file_packager_async_part3 branch August 8, 2025 21:11
sbc100 added a commit to sbc100/emscripten that referenced this pull request Aug 8, 2025
sbc100 added a commit to sbc100/emscripten that referenced this pull request Aug 8, 2025
sbc100 added a commit to sbc100/emscripten that referenced this pull request Aug 8, 2025
sbc100 added a commit to sbc100/emscripten that referenced this pull request Aug 8, 2025
sbc100 added a commit to sbc100/emscripten that referenced this pull request Aug 8, 2025
sbc100 added a commit to sbc100/emscripten that referenced this pull request Aug 8, 2025
sbc100 added a commit to sbc100/emscripten that referenced this pull request Aug 9, 2025
sbc100 added a commit to sbc100/emscripten that referenced this pull request Aug 11, 2025
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