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

Test redundant varint fields decoding in JS #1530

Closed
goburns opened this issue May 13, 2016 · 1 comment
Closed

Test redundant varint fields decoding in JS #1530

goburns opened this issue May 13, 2016 · 1 comment

Comments

@goburns
Copy link

goburns commented May 13, 2016

Following up internal conversation we want to add test into JS protobuf. It's in order to ensure that parser is okay to process non-canonical varints e.g. 8C 80 80 80 00 instead of just 0C. There are some cases when it's useful.
Pull request: #1529
@goburns
Copy link
Author

goburns commented May 14, 2016

Merged

@goburns goburns closed this as completed May 14, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@goburns