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.

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

Verify correct escaping in use of Turtle and SPARQL tokens #55

Open
kasei opened this issue Jan 11, 2016 · 2 comments
Open

Verify correct escaping in use of Turtle and SPARQL tokens #55

kasei opened this issue Jan 11, 2016 · 2 comments
Assignees
@kasei

Comments

@kasei
Copy link
Owner

kasei commented Jan 11, 2016

I suspect that the $token->value values are sometimes (or perhaps always) being stored in an escaped form (e.g. when parsing Turtle unicode escapes like \u0063). This needs to be fixed, so that the stored value is always the actual unicode string intended. If this requires changes, then the Turtle and SPARQL serializers need to be verified as re-encoding where necessary.
@kasei
Copy link
Owner Author

kasei commented Jan 11, 2016

Also consider changing the LANG token at the same time to drop the "@". Similar tokens like VAR do not keep their sigil in the stored value.

@kasei
Copy link
Owner Author

kasei commented Jan 19, 2016

Simple tests for Turtle added in c04b64b.

kasei added a commit that referenced this issue Jan 20, 2016
@kasei
Added simple SPARQL parsing tests (#55).
fb72f52
kasei added a commit that referenced this issue Jul 28, 2016
@kasei
Added tests for turtle parser escape handling (#55).
f37383f
@kasei kasei assigned kasei and unassigned kasei Jan 9, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@kasei kasei

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@kasei