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

[wasm] clean up js argument parsing #98537

Closed
wants to merge 9 commits into from
Closed

Conversation

lewing
Copy link
Member

@lewing lewing commented Feb 16, 2024

The goal here is to reference as little managed string manipulation as possible and avoid unnecessary copies.

@ghost ghost assigned lewing Feb 16, 2024
@lewing lewing changed the title [wasm] Reduce the allocations in parsing the js [wasm] clean up js argument parsing Feb 16, 2024
@lewing lewing closed this Feb 21, 2024
@lewing lewing reopened this Feb 21, 2024
@lewing lewing closed this Feb 21, 2024
@github-actions github-actions bot locked and limited conversation to collaborators Mar 22, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant