Skip to content

PARSEC-5223 - Add various fixes for WASM app#175

Open
mtrang1263 wants to merge 10 commits intostablefrom
mt-fix-webapp
Open

PARSEC-5223 - Add various fixes for WASM app#175
mtrang1263 wants to merge 10 commits intostablefrom
mt-fix-webapp

Conversation

@mtrang1263
Copy link

@mtrang1263 mtrang1263 commented Mar 4, 2026

Fixes

  1. Audio not working when connecting to a host.
  2. Fixed issue with destroying non-main threads. This caused an issue when attempt to destroy the audio threads introduced in this PR: https://github.com/parsec-cloud/parsec/pull/2092

This PR relies on: https://github.com/WebAssembly/js-promise-integration/blob/main/proposals/js-promise-integration/ImplementationGuide.md#jspi-implementation-notes to correctly implement MTY_RunAndYield. This is available on chrome browsers since 137 which was released in May 2025.

We do have fallbacks to the old behavior that does not consume excessive memory but also does not support the cleanup of streams.

Blocks: https://github.com/parsec-cloud/parsec/pull/2063

@mtrang1263 mtrang1263 requested a review from a team as a code owner March 4, 2026 23:31
@mtrang1263 mtrang1263 added the 2. Review requested You have coded your code, and can now be reviewed. label Mar 5, 2026
@Namaneo Namaneo added 1. Coding Feature has been described inside of the parsec branch, and you are coding on it. and removed 2. Review requested You have coded your code, and can now be reviewed. labels Mar 6, 2026
@Namaneo Namaneo added 2. Review requested You have coded your code, and can now be reviewed. and removed 1. Coding Feature has been described inside of the parsec branch, and you are coding on it. labels Mar 6, 2026
@mtrang1263 mtrang1263 requested a review from Kodikuu March 12, 2026 21:44
if (!MTY.ccanvas) {
MTY.ccanvas = document.createElement('canvas');
MTY.cctx = MTY.ccanvas.getContext('2d');
MTY.cctx = MTY.ccanvas.getContext('2d', {"willReadFrequently": true});
Copy link
Author

Choose a reason for hiding this comment

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

Resolve warning we were receiving with the cursor.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

2. Review requested You have coded your code, and can now be reviewed.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants