-
Notifications
You must be signed in to change notification settings - Fork 858
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
Grammar appears to allow character forbidden by docs #1013
Comments
You're comparing different versions of the standard. If you look at the current version: and the current ABNF: you will see that This was a change that we made explicit in #924. It could be reverted if we merged #996, which I'm personally opposed to doing. So keep an eye out for this. By the time v1.1.0rc1 is released, this matter will be resolved, one way or another. |
Am I? I'm comparing the specification 1.0.0:
To the grammar tagged 1.0.0
If it's going to be fixed in a later version then it's not a big deal, but it does seem to be a real mistake in one of the two documents. |
Indeed! I'm pretty sure the intent was to forbid "Control characters other than tab", like the written spec says, so the ABNF is wrong in allowing U+007F. But in any case it's a bug in one of the spec documents and the next version must fix it. So it's good that the current mainline already fixed it by allowing nearly all control chars. |
toml/toml.abnf
Line 39 in 8eae5e1
The grammar for valid comment characters includes the class
U+0020 - U+007F
. The docs, however, forbid the use ofU+007F
in comments.The text was updated successfully, but these errors were encountered: