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

Fix off-by-one loop index #186

Merged
merged 1 commit into from
Mar 25, 2024
Merged

Fix off-by-one loop index #186

merged 1 commit into from
Mar 25, 2024

Conversation

jyavenard
Copy link
Member

@jyavenard jyavenard commented Mar 19, 2024

(fixes #184)

  • Tests are written and can be reviewed and commented upon at:
    • No need for new tests, the wpt already use the proper algorithm.

(See WHATWG Working Mode: Changes for more details.)


Preview | Diff

@annevk
Copy link
Member

annevk commented Mar 19, 2024

@padenot can you help review?

@padenot
Copy link
Collaborator

padenot commented Mar 19, 2024

Looks good, sorry for the mistake in the first place.

@annevk annevk merged commit 975191b into whatwg:main Mar 25, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

Parse a Vint algorithm is incorrect due to off by one loop.
3 participants