\p{IsLatin} is used and does not work on Android

[arnt]

LanguageDetector.kt line 412 uses \p{IsLatin}, which definitely ought to work on Android, but doesn't. Using \p{Latin} works, so I suggest you do that. The next two lines probably have the same issue.

Looks great BTW, and I love that you included Latin already.

[pemistahl]

Hello @arnt, thank you for using my library and letting me know about this problem.

I'm going to fix this bug in the upcoming release 0.4.0. I still need some time to finish it but it won't take long anymore.

[pemistahl]

I've just fixed this problem with commit fd52216.

Interestingly enough, I get a PatternSyntaxException on my developer machine with the standard JDK 8 using the character classes without the "Is-" prefix. Anyway, now both notation variants are tried in order, so one of them ought to work.

I will close this issue once release 0.4.0 is completed.

Thanks again for your contribution. Very much appreciated.

[arnt]

That was quick!

I looked more thoroughly at the documentation, and it seems that the misunderstanding in the SO question is that "latin" isn't actually a unicode category. One might easily think so, but it's not. Sigh. IMO both the Java documentation and the Android implementation could do with some improvement.

Anyway. Such is life. Thanks for the quick fix.