Skip to content

Conversation

@faddat
Copy link
Contributor

@faddat faddat commented Dec 3, 2024

I noticed that the pointer issue is not resolved, so I absolutely must, in the strongest possible terms, urge you to merge this pull request because the issue is still there.

For these tests, passing 95% of the time should be treated as passing 0% of the time.

note

I noticed that the references came from wasmvm. So I am making this PR be the actual test, and it will use the latest wasmvm from main.

If this runs the test incorrectly, please let me know.

Frankly, I don't trust wasmvm, and never have, due to its use of cgo, which is nigh on impossible to debug.

@chipshort
Copy link
Contributor

As you can see, the fix works when you use the correct wasmvm version that includes the fix.

@faddat
Copy link
Contributor Author

faddat commented Dec 4, 2024

Indeed. Sorry I forgot to close this :)

It totally does work and that's super

gday

@faddat faddat closed this Dec 4, 2024
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