Re: RFR: 8248655: Support supplementary characters in String case insensitive operations

2020-07-17 Thread naoto . sato
Hi, Based on the suggestions, I modified the fix as follows: https://cr.openjdk.java.net/~naoto/8248655.8248434/webrev.01/ Changes from the initial revision are: - Shared the implementation between compareToCI() and regionMatchesCI() - Enabled immediate short cut if two code points match. - Cr

Re: RFR: 8248655: Support supplementary characters in String case insensitive operations

2020-07-17 Thread Mark Davis ☕️
One option is to have a fast path that uses char functions, up to the point where you hit a high surrogate, then drop into the slower codepoint path. That saves time for the high-runner cases. On the other hand, if the times are good enough, you might not need the complication. Mark On Fri, Jul