[
https://issues.apache.org/jira/browse/LUCENE-6760?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14708179#comment-14708179
]
Uwe Schindler commented on LUCENE-6760:
---------------------------------------
OK Robert, thanks for the opinion. I will go for the first patch and simply
remove the non-whitespace character.
FYI, I checked with Eclipse call trace: The method is only used by this single
test, so it's not really a useful utility method in TestUtil. We should move it
to the test. But I will for now only fix the problem, so the recent Java 9
builds pass!
> TestUtil#randomWhitespace() broken for Java 9 (Unicode 7)
> ---------------------------------------------------------
>
> Key: LUCENE-6760
> URL: https://issues.apache.org/jira/browse/LUCENE-6760
> Project: Lucene - Core
> Issue Type: Test
> Environment: Java 9 build 76+
> Reporter: Uwe Schindler
> Labels: Java9
> Attachments: LUCENE-6760.patch, LUCENE-6760.patch
>
>
> Java 9 changed its character tables to unicode version 7.0. The table updates
> are listed here: http://hg.openjdk.java.net/jdk9/jdk9/jdk/rev/86206517258d
> Because of this, character {{\u180E}} is no longer treated as whitespace, so
> TestUtil#randomWhitespace() fails.
> I will remove this character from the list and update the documentation
> JRE_VERSION_MIGRATION.txt.
> We should maybe make this character list dynamic (e.g. TestUtil initializes
> it on static class init one time by iterating over all 16 bit characters).
> Maybe somebody else has an idea (there is already a TODO in the code about
> that).
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
