Skip to content
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

Jump to bottom

use yencode instead of base64 #213

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

use yencode instead of base64 #213

wants to merge 2 commits into from

Conversation

jeetiss
Copy link

@jeetiss jeetiss commented Feb 3, 2023

Hello folks.

I made this PR to test the yenc encoding

It looks like it produces a bigger encoded string, but it is compressed much better with gzip.

stats for pkg3d/rapier.es.js:

*yEnc*
Bundle Size:  2.16 MB           
Minified Size:  1.45 MB     
Gzipped Size:  563.37 KB 

*base-64*
Bundle Size:  1.89 MB
Minified Size:  1.89 MB
Gzipped Size:  707.67 KB

Feel free to close this PR

# for free to join this conversation on GitHub. Already have an account? # 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
@jeetiss