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

Eager/lazy ranges? #10

Open
s-ludwig opened this issue Apr 26, 2015 · 1 comment
Open

Eager/lazy ranges? #10

s-ludwig opened this issue Apr 26, 2015 · 1 comment

Comments

@s-ludwig
Copy link
Collaborator

Currently, JSONLexerRange eagerly skips whitespace for performance reasons, but everything else is evaluated lazily. The problem with eager WS skipping is that it makes the lexer unusable in a request/response scenario where a requests consists of a single JSONValue. Skipping the whitespace at the end would block on the connection until the next request arrives, which may never happen if the previous one never gets processed.

This should either be made configurable, or the fully lazy version needs to be brought up to the same speed as the eager version.

@Panke
Copy link

Panke commented Dec 12, 2020

Is this still a problem?

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

No branches or pull requests

2 participants