Skip to content

Escape character text #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.

Sign up for GitHub

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

Jump to bottom
Closed
afs opened this issue Feb 12, 2023 · 3 comments
Closed

Escape character text #5

afs opened this issue Feb 12, 2023 · 3 comments
Labels
propose closing Proposed for closing

Comments

@afs
Copy link
Contributor

afs commented Feb 12, 2023 •
edited
Loading

The doc says:

"""
Literals delimited by ' (U+0027), may not contain the characters ', LF (U+000A), or CR (U+000D).
Literals delimited by ", may not contain the characters ", LF, or CR.
Literals delimited by ''' may not contain the sequence of characters '''.
Literals delimited by """ may not contain the sequence of characters """.
"""

It should say

"""
Literals delimited by ' (U+0027), may not contain an unescaped ' character, ...
"""

or

"""
Literals delimited by ' (U+0027) must use an escape sequence to include the ' character,
"""

Similarly for "-strings and for triple quoted strings (\''' is possible).
@afs afs mentioned this issue Feb 12, 2023
Merged
@gkellogg
Copy link
Member

Presumably keeping the restriction on LF and CR in each.

@afs
Copy link
Contributor Author

afs commented Feb 13, 2023

Yes - it's the ... above.

gkellogg added a commit that referenced this issue Feb 13, 2023
@gkellogg
Change literal quote requirements described in #5.
4024306
@gkellogg gkellogg mentioned this issue Feb 13, 2023
Merged
gkellogg added a commit that referenced this issue Feb 15, 2023
@domel @gkellogg
Typos and minor changes (#6)
14f376b 
* update example 26
  add missing triple
* update examples 24 and 26
  add missing `@prefix`
* Change literal quote requirements described in #5.
* Fix typo reported in erratum 11.
* Abbreviation for SPARQL is updated. This was reported in erratum 26.

---------

Co-authored-by: Gregg Kellogg <gregg@greggkellogg.net>
@gkellogg
Copy link
Member

This was addressed in 4024306 and can probably be closed.

@gkellogg gkellogg added the propose closing Proposed for closing label Jun 10, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
propose closing Proposed for closing
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@gkellogg @afs