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

Add UNICODE_CHARACTER_CLASS flag to Regex flag parsing #2895

Closed
bleskes opened this Issue Apr 15, 2013 · 1 comment

Comments

Projects
None yet
2 participants
@bleskes
Member

bleskes commented Apr 15, 2013

Hi,

Java's Regex have issue with character classes when it comes to unicode content (
http://stackoverflow.com/questions/4304928/unicode-equivalents-for-w-and-b-in-java-regular-expressions
)

Apparently they solved things in Java7 by adding a special flag: UNICODE_CHARACTER_CLASS ( http://docs.oracle.com/javase/7/docs/api/java/util/regex/Pattern.html#UNICODE_CHARACTER_CLASS )

Sadly org.elasticsearch.common.regex doesn't "understand" it. As Java7 is the recommended runtime for ES I think it's OK to add it even if not supported by earlier JDK.

Cheers,
Boaz

@ghost ghost assigned s1monw Apr 16, 2013

@s1monw

This comment has been minimized.

Show comment
Hide comment
@s1monw

s1monw Apr 16, 2013

Contributor

yeah makes sense... I will add it in a bit

Contributor

s1monw commented Apr 16, 2013

yeah makes sense... I will add it in a bit

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment