Skip to content

Escaped surrogates #3282

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

Merged
merged 7 commits into from
Jun 30, 2025
Merged

Escaped surrogates #3282

merged 7 commits into from
Jun 30, 2025

Conversation

afs
Copy link
Member

@afs afs commented Jun 25, 2025

GitHub issue resolved #3281
and a few clean-ups.

This implements the simpler separate check of RDF string in TokenizerText, not the more complicated check described in #3281.


  • Tests are included.
  • Documentation change and updates are provided for the Apache Jena website
  • Commits have been squashed to remove intermediate development commit messages.
  • Key commit messages start with the issue number (GH-xxxx)

By submitting this pull request, I acknowledge that I am making a contribution to the Apache Software Foundation under the terms and conditions of the Contributor's Agreement.

@afs afs marked this pull request as draft June 26, 2025 08:46
@afs afs force-pushed the escaped-surrogates branch 3 times, most recently from 5979783 to ce94b4c Compare June 26, 2025 17:45
@afs afs force-pushed the escaped-surrogates branch 2 times, most recently from 03252e3 to d6e27a9 Compare June 27, 2025 08:22
@afs afs marked this pull request as ready for review June 27, 2025 08:22
@afs afs force-pushed the escaped-surrogates branch 5 times, most recently from b96dac4 to b057fad Compare June 28, 2025 20:51
@afs afs force-pushed the escaped-surrogates branch from b057fad to 43b322e Compare June 30, 2025 06:39
@afs afs merged commit d954ed5 into apache:main Jun 30, 2025
@afs afs deleted the escaped-surrogates branch June 30, 2025 11:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Reject surrogate codepoints that are \u or \U escaped.
1 participant