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

Fixed rust examples #20

Merged
merged 1 commit into from Oct 2, 2018
Merged

Fixed rust examples #20

merged 1 commit into from Oct 2, 2018

Conversation

vlad20012
Copy link
Contributor

Rust 1.29 brings stable allocator API.
Outdated API removed from the rust stdlib.

I still can't compile "rust-regex" due to
"Error in command 'compile': Method code too large!"

Rust 1.29 brings stable allocator API.
Outdated API removed from the rust stdlib.

I still can't compile "rust-regex" due to
"Error in command 'compile': Method code too large!"
@cretz
Copy link
Owner

cretz commented Aug 7, 2018

Thanks! Will review in the next couple of days.

I still can't compile "rust-regex" due to "Error in command 'compile': Method code too large!"

Yeah, some runtimes are making really large WASM functions which exceed the JVM method size limit. I started work on this in #19 but I think I will have to change tack. I had an algorithm that tried to do an intelligent de-dupe, but I think now I'm just gonna break the method in half.

@vlad20012
Copy link
Contributor Author

ping

@cretz
Copy link
Owner

cretz commented Sep 30, 2018

Sorry, been busy. I did complete #19 a couple weeks ago. Will update when I've worked with and tested this.

@cretz cretz merged commit 8a9eaf2 into cretz:master Oct 2, 2018
@cretz
Copy link
Owner

cretz commented Oct 2, 2018

Merged into master, but I have not done any tests yet. Doing now.

@cretz
Copy link
Owner

cretz commented Oct 2, 2018

I am still working through this (finding bugs in my method splitter). Could take some time.

@cretz
Copy link
Owner

cretz commented Oct 4, 2018

Ok, this should now be fixed in master

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.

None yet

2 participants