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

Improve in rule repetition re-sync recovery. #133

Closed
bd82 opened this issue Mar 6, 2016 · 0 comments
Closed

Improve in rule repetition re-sync recovery. #133

bd82 opened this issue Mar 6, 2016 · 0 comments

Comments

@bd82
Copy link
Member

bd82 commented Mar 6, 2016

The repetition re-sync recovery can sync to another iteration of the repetition.
but not to the originally expected token immediately following the repetition.

so:

{
"key1" : 1 // missing first comma
"key2" : 2,
"key3" : 3
}

Will successfully recover as it will sync to the next comma.
But:

{
"key1" : 1,
"key2" : 2 // missing second comma
"key3" : 3
}

Will fail re-syncing as it has no comma to re-sync to.

The improvement is to allow re-syncing to the token originally expected after the repetition.
(in the above example this would be "}")

@bd82 bd82 closed this as completed in 71c3516 Mar 6, 2016
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

1 participant