You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@lucenenet.apache.org by GitBox <gi...@apache.org> on 2020/07/22 11:47:01 UTC

[GitHub] [lucenenet] NightOwl888 opened a new pull request #316: Fix for NGram TestUTF8FullRange() tests (See #269)

NightOwl888 opened a new pull request #316:
URL: https://github.com/apache/lucenenet/pull/316


   The problem was that the code point was being cast to a `char` before calling the `IndexOf` method, which means that it didn't support surrogate pairs. In rare instances where the cast turned the code point into a valid token character the test failed.
   
   .NET doesn't have a built-in overload of `String.IndexOf()` that accepts a code point, that is an extension method in J2N.


----------------------------------------------------------------
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



[GitHub] [lucenenet] NightOwl888 merged pull request #316: Fix for NGram TestUTF8FullRange() tests (See #269)

Posted by GitBox <gi...@apache.org>.
NightOwl888 merged pull request #316:
URL: https://github.com/apache/lucenenet/pull/316


   


----------------------------------------------------------------
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