RFR: 8241055: Regex Grapheme Matcher Performance Depends too much on Total Input Sequence Size

Philipp Kunz philipp.kunz at paratix.ch
Tue Apr 14 16:26:54 UTC 2020


Hi Naoto,

I agree, see attached patch.

Regards,
Philipp


On Thu, 2020-03-26 at 14:14 -0700, naoto.sato at oracle.com wrote:
> Hi Philipp,
> 
> I looked at the patch, and it looks good to me. As to the test case, 
> since it is measuring the performance, I would make it in a JMH test. 
> Maybe you would want to put it similar to 
> open/test/micro/org/openjdk/bench/java/util/regex/PatternBench.java
> 
> Minor suggestion: I would rather rename "getGraphemeType()" to just 
> "getType()", as it is simply retrieving the character type.
> 
> Naoto
> 
> On 3/21/20 7:58 AM, Philipp Kunz wrote:
> > Hi,
> > 
> > Any opinions on the attached patch or someone tempted to sponsor it?
> > 
> > Regards,
> > Philipp
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 20200414-bug8241055-graphemecomplexity.patch
Type: text/x-patch
Size: 14816 bytes
Desc: not available
URL: <https://mail.openjdk.java.net/pipermail/core-libs-dev/attachments/20200414/b4345bed/20200414-bug8241055-graphemecomplexity-0001.patch>


More information about the core-libs-dev mailing list