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