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

Refactor JsonReader's consumeNonExecutePrefix method #1505

Merged

inder123 merged 1 commit into google:master from Degubi:jsonreader_inlineprefix

Oct 4, 2019

Contributor

Degubi commented Mar 31, 2019 •

edited

Loading

Remove static array in JsonReader.
Replace loop if-s with 5 explicit if statements, each checking 1 character at a time.


          Refactor prefix array & loop logic, use if-s instead

ef2def3

googlebot added the cla: yes label

inder123 self-assigned this

inder123 merged commit 9bf25c2 into google:master

Degubi deleted the jsonreader_inlineprefix branch

October 5, 2019 05:56

delvh mentioned this pull request

(Not a bug) Corrupted JSON file with Files.writeString #1919

Closed

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

Labels