[GitHub] [jena] afs commented on a change in pull request #815: JENA-1981 Unicode and SUBSTR

[GitBox <gi...@apache.org>]

afs commented on a change in pull request #815:
URL: https://github.com/apache/jena/pull/815#discussion_r508694950



##########
File path: jena-arq/src/test/java/org/apache/jena/sparql/expr/TestFunctions2.java
##########
@@ -165,6 +167,23 @@
     public void substr_21()         { test("substr('', 'one', 3)",            "''") ; }
     @Test(expected=ExprEvalException.class)
     public void substr_22()         { test("substr('', 1, 'three')",            "''") ; }
+    
+    // Codepoint outside UTF-16. 
+    // These are  U+0001F46A 👪 - FAMILY

Review comment:
       :-)
   
   I didn't put them raw in the tests themselves because of portability concerns over binary/text.
   I did test with 👪.
   (ARQ is not handling \U in SPARQL properly yet.)
   




----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: pr-unsubscribe@jena.apache.org
For additional commands, e-mail: pr-help@jena.apache.org