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

The small symbolization #2573

Draft
wants to merge 6 commits into
base: master
Choose a base branch
from
Draft

The small symbolization #2573

wants to merge 6 commits into from

Conversation

hmdne
Copy link
Member

@hmdne hmdne commented Aug 7, 2023

Ref: #2144

This PR will explore ways to gradually move to new API that sets method keys using JavaScript symbols and make that API available before we do the full migration.

@hmdne
Copy link
Member Author

hmdne commented Aug 7, 2023

 Comparison of the Asciidoctor (a real-life Opal application) compile and run:
                  Compile time: 4.536 (+25.22%) -> 4.490 (+24.91%) (change: -1.01%)
                      Run time: 1.224 (+8.87%) -> 1.223 (+36.85%) (change: -0.02%)
                   Bundle size: 4625.64 kB -> 4718.50 kB (change: +2.01%)
          Minified bundle size: 990.33 kB -> 1074.65 kB (change: +8.51%)
            Mangled & minified: 698.46 kB -> 687.68 kB (change: -1.54%)

The performance is almost the same (even though we haven't introduced real Symbols yet). Bundle size increase for minified is expected.

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

1 participant