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.

Sign up for GitHub

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

Jump to bottom

Fixing the size of Rust binaries #745

Closed
wants to merge 8 commits into from
Closed

Fixing the size of Rust binaries #745

wants to merge 8 commits into from

Conversation

Ztry8
Copy link
Contributor

@Ztry8 Ztry8 commented Jul 29, 2024 •
edited
Loading

The size before the fix was 496 bytes! (without wasm-opt)
The size is now 286 bytes! (also without wasm-opt)

@Ztry8 Ztry8 closed this Jul 29, 2024
@Ztry8
Copy link
Contributor Author

Ztry8 commented Jul 29, 2024 •
edited
Loading

Fixed vecs now! See also
A snake tutorial fix is already in the works!
Please don't merge yet!

@Ztry8 Ztry8 reopened this Jul 29, 2024
@Ztry8
Copy link
Contributor Author

Ztry8 commented Jul 29, 2024

In summary, the following changes were made:

  • The size without wasm opt was reduced from 496 to 293, which represents a 60% reduction!
  • The dependencies were removed and, upon testing, appear to be functioning correctly without them!
  • The snake game tutorial has been updated!

Thank you for this amazing and very cool library!

@Ztry8
Copy link
Contributor Author

Ztry8 commented Jul 29, 2024

The size did not change after using wasm-opt, maybe it is no longer needed and need to update the documentation here

@Ztry8 Ztry8 closed this Aug 1, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@Ztry8