Looks good.
As you noticed, the error does not seem to occur with JDK14 as bootjdk.
I will push it after I confirmed OK on my side. Thank you for your
contribution.
Naoto
On 4/16/20 12:26 PM, Philipp Kunz wrote:
Hi Naoto,
Changed as you suggested.
However, I could not verify it due to, mos
Hi Naoto,
Changed as you suggested.However, I could not verify it due to, most
probably totally unrelated, Error: Resolution failed: Module
jdk.scripting.nashorn.shell does not read a module that exports
jdk.internal.editor.spi.
Regards,Philipp
On Wed, 2020-04-15 at 11:14 -0700, naoto.s...@oracle.
Hi Philipp,
Thanks for the update. I like you enriched the RegExTest.java. Two nits:
- RegExTest.java at line 4805: I'd use parentheses for this return
statement.
- PatternBench.java should keep the original copyright year, i.e.,
"2020," -> "2019, 2020,"
Otherwise, it looks good.
Naoto
O
Hi Naoto,
I agree, see attached patch.
Regards,
Philipp
On Thu, 2020-03-26 at 14:14 -0700, naoto.s...@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 w
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 rathe
I am planning to work on it.
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
Hi,
Any opinions on the attached patch or someone tempted to sponsor it?
Regards,
Philipp
diff -r 84215fa115fc src/java.base/share/classes/java/util/regex/Grapheme.java
--- a/src/java.base/share/classes/java/util/regex/Grapheme.java Fri Mar 20 17:37:52 2020 -0700
+++ b/src/java.base/share/classes