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.

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

Roll StreamParserAdapter into the StreamParser #83

Closed
stig opened this issue Jul 1, 2011 · 1 comment
Closed

Roll StreamParserAdapter into the StreamParser #83

stig opened this issue Jul 1, 2011 · 1 comment
Labels
Rejected

Comments

@stig
Copy link
Collaborator

stig commented Jul 1, 2011

Not only is the myriad of classes confusing, it should also be easier to eek some more performance out of the parser if we don't have to go through so many instantiations + message sends.
@stig stig closed this as completed Jul 1, 2011
everyscape pushed a commit to everyscape/json-framework that referenced this issue Aug 15, 2011
@stig
Roll SBJsonStreamParserAdapter into SBJsonStreamParser. Fixes SBJson#83.
405bbec
@stig
Copy link
Collaborator Author

stig commented Aug 28, 2011

Backed out this change. People are using the low-level protocol, and I couldn't find a more elegant way of implementing it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Rejected
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@stig