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

Fix undefined being emitted instead of null #7112

Merged
merged 1 commit into from
Oct 20, 2024
Merged

Conversation

cknitt
Copy link
Member

@cknitt cknitt commented Oct 20, 2024

No description provided.

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Syntax Benchmarks

Benchmark suite Current: a1351ef Previous: e1b7fb7 Ratio
Parse RedBlackTree.res - time/run 1.2423180933333333 ms 1.2123143266666667 ms 1.02
Parse RedBlackTree.res - allocs/run 199057 words 199057 words 1
Print RedBlackTree.res - time/run 2.307105593333333 ms 2.3104395133333333 ms 1.00
Print RedBlackTree.res - allocs/run 236403 words 236403 words 1
Print RedBlackTreeNoComments.res - time/run 2.0943465266666665 ms 2.10057036 ms 1.00
Print RedBlackTreeNoComments.res - allocs/run 249186 words 249186 words 1
Parse Napkinscript.res - time/run 39.13450264666667 ms 39.28006235333333 ms 1.00
Parse Napkinscript.res - allocs/run 8670066 words 8670066 words 1
Print Napkinscript.res - time/run 76.31320972666666 ms 77.00100409999999 ms 0.99
Print Napkinscript.res - allocs/run 9776327 words 9776327 words 1
Parse HeroGraphic.res - time/run 5.09736366 ms 5.13472718 ms 0.99
Parse HeroGraphic.res - allocs/run 1219326 words 1219326 words 1
Print HeroGraphic.res - time/run 8.68004098 ms 8.775952553333333 ms 0.99
Print HeroGraphic.res - allocs/run 1396466 words 1396466 words 1

This comment was automatically generated by workflow using github-action-benchmark.

@cknitt cknitt merged commit 6d9df0f into master Oct 20, 2024
20 checks passed
@cknitt cknitt deleted the fix-null-undefined branch October 20, 2024 19:06
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