Skip to content

Revert "Optimize JSON parsing a bit"#14804

Merged
straight-shoota merged 1 commit intomasterfrom
revert-14366-optimize-json
Jul 11, 2024
Merged

Revert "Optimize JSON parsing a bit"#14804
straight-shoota merged 1 commit intomasterfrom
revert-14366-optimize-json

Conversation

@straight-shoota
Copy link
Member

Reverts #14366

Resolves #14803

@straight-shoota straight-shoota added topic:stdlib:serialization kind:regression Something that used to correctly work but no longer works labels Jul 10, 2024
@straight-shoota straight-shoota added this to the 1.13.1 milestone Jul 10, 2024
@straight-shoota straight-shoota merged commit 2204be7 into master Jul 11, 2024
@straight-shoota straight-shoota deleted the revert-14366-optimize-json branch July 11, 2024 08:19
@straight-shoota straight-shoota added the kind:bug A bug in the code. Does not apply to documentation, specs, etc. label Jul 12, 2024
@kuraga
Copy link

kuraga commented Jul 12, 2024

Has been #14803 test case added?

@straight-shoota
Copy link
Member Author

Not yet. It isn't essential for the 1.13.1 release because we know it works.
Good point to add it though. Would you mind making a PR?

@kuraga
Copy link

kuraga commented Jul 15, 2024

Sorry, no. (Don't use Crystal.)
But I see that this revert removes some specs and and doesn't add some from #14803 :) .

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

kind:bug A bug in the code. Does not apply to documentation, specs, etc. kind:regression Something that used to correctly work but no longer works topic:stdlib:serialization

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1.13.0 - a big problem with UTF strings after JSON.parse when some special symbols present; broken encoding

4 participants