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

invalid-reverse-solidus error definition has character mixed up. #772

Open
JamesNJep opened this issue May 26, 2023 · 3 comments
Open

invalid-reverse-solidus error definition has character mixed up. #772

JamesNJep opened this issue May 26, 2023 · 3 comments

Comments

@JamesNJep
Copy link

Under the host passing section https://url.spec.whatwg.org/#host-parsing

What I think it is trying to say is that the URL scheme uses the solidus character U+002F (/) and not the reverse solidus character U+005C (), but the characters seem to be in the wrong order in the sentence above the example.

@annevk
Copy link
Member

annevk commented May 26, 2023

Are you talking about https://url.spec.whatwg.org/#invalid-reverse-solidus?

The characters after the scheme are correct, but those are not the ones in error. The ones later on are.

@JamesNJep
Copy link
Author

Yes was talking about https://url.spec.whatwg.org/#invalid-reverse-solidus thank you for looking at this.

The part I had trouble understanding is the description sentence above the example.
The description text reads

"The URL has a special scheme and it uses U+005C () instead of U+002F (/)."

The way I have read and interpreted this when first reading this is that the sentence is making a statement, about how the URL should be written to avoid this error, but I think the sentence is actually trying to make a positive statement to describe the error condition.

I've re-read this table several times and I think the other items in the table are easier to understand because they use words or prases like, "exceeds", "improper", "too many", "too few", "too big", "invalid" that make it clear that the statement is describing an error state.

I think if the sentence was written more like the desciption for https://url.spec.whatwg.org/#example-host-missing and used "but" instead of "and" that would make it easier to understood the statment was describing the error condition if someone is jumping directly to that description.
For example if it was written like this

"The URL has a special scheme, but has used U+005C () in the place of U+002F (/)."

@annevk
Copy link
Member

annevk commented Jun 3, 2023

I'm not sure how to apply that consistently. Many of these are statements describing the error state in exactly that fashion, e.g.,

An IPv4 address ends with a U+002E (.).

Although, maybe we should remove "instead of U+002F (/)" or put that between parenthesis? Nothing else has advice like that so maybe removing it would be best and make it clearer overall.

What do you think?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

No branches or pull requests

2 participants