-
Notifications
You must be signed in to change notification settings - Fork 5
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
Specify or forbid whitespace normalisation #24
Comments
This was already raised by David on the CDs repository. I think we should clarify this in the standard and the normative RNCs (actually everywhere). Assigning David and James to come up with a pull request. |
The original OM3 issue says:
I think the initial comment is not correct actually. The existing examples with spaces are valid. The standard in words says CDName contains the name and for syntactic requirements explicitly defers to the relax schema, that says
which implies that the content is tokenized including white space normalisation and the resulting token checked against the NCName production. so
is valid and equivalent to
and
is invalid. So no change is needed but personally I'd be quite happy to remove the spaces from the core CDs they may not be invalid but they are not a good example to follow. The same implicit tokenisation applies to That said, just because a change isn't needed doesn't mean an explanatory note couldn't be added somewhere. |
I think that is the least we should do. The spaces are indeed a very bad practice. Would you? |
I think this would be helpful, and also say that the spaces around are "bad practice" (deprecated). Would you make a pull request of this? |
see OpenMath/OM3#134
The text was updated successfully, but these errors were encountered: