Skip to content

Fix some calls to free() missing alignment #3639

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Merged
merged 1 commit into from
Oct 3, 2023

Conversation

daxpedda
Copy link
Collaborator

@daxpedda daxpedda commented Oct 3, 2023

Apparently I missed some calls to free() in #3463.
@Liamolucko would appreciate if you'd take a look as well.

Thanks @Polish-Civil.
Fixes #3638.

@daxpedda daxpedda requested a review from Liamolucko October 3, 2023 10:24
Copy link
Collaborator

@Liamolucko Liamolucko left a comment

Choose a reason for hiding this comment

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

LGTM!

@daxpedda daxpedda merged commit 9e80eb2 into rustwasm:main Oct 3, 2023
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Alignment parameter is not passed to __wbindgen_free in js bindings.
2 participants