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

[css-syntax] Problem with "Consume a name" #3628

Closed
tabatkins opened this issue Feb 7, 2019 · 0 comments
Closed

[css-syntax] Problem with "Consume a name" #3628

tabatkins opened this issue Feb 7, 2019 · 0 comments
Labels
Closed Accepted as Obvious Bugfix Commenter Satisfied Commenter has indicated satisfaction with the resolution / edits. css-syntax-3 Testing Unnecessary Memory aid - issue doesn't require tests

Comments

@tabatkins
Copy link
Member

(migrated from the mailing list)

Johannes Koch said:

ED 2014-03-21, section 4.3.11 Consume a name:

When the algorithm is based on the consumed next token, at the end of
this section (when returning the name), the current input code point is
the first after the name. This code point will be missed when e.g.
returning to 4.3.1 Consume a token, which again consumes the next token.

So. e.g., in 4.3.11 the first non-name code point could be reconsumed
before returning the name.


Tab Atkins said:

Thanks for yet another detailed catch! Fixed.

@tabatkins tabatkins added css-syntax-3 Closed Accepted as Obvious Bugfix Commenter Satisfied Commenter has indicated satisfaction with the resolution / edits. Needs Testcase (WPT) labels Feb 7, 2019
@tabatkins tabatkins added Testing Unnecessary Memory aid - issue doesn't require tests and removed Needs Testcase (WPT) labels Feb 15, 2019
@tabatkins tabatkins added this to the CSS Syntax 3 June 2019 CR milestone Jun 28, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Closed Accepted as Obvious Bugfix Commenter Satisfied Commenter has indicated satisfaction with the resolution / edits. css-syntax-3 Testing Unnecessary Memory aid - issue doesn't require tests
Projects
None yet
Development

No branches or pull requests

1 participant