Hi Severin,
On 22/05/2019 17:34, Severin Gehwolf wrote:
> Bug: https://bugs.openjdk.java.net/browse/JDK-8222737
> webrev: http://cr.openjdk.java.net/~sgehwolf/webrevs/JDK-8222737/02/
> (includes changes to top/hotspot/jdk/langtools repos)
>
> Example excerpt from a run:
> http://cr.openjdk.java.net/~sgehwolf/webrevs/JDK-8222737/02/example_output.txt
>
> Thoughts?
Unfortunately, my makefile fu is too limited to provide a reliable
review of the patch (I'm mostly just surprised it took so few changes to
achieve). However, I can still wholeheartedly applaud this endeavour and
its present outcome.
As you say tuning the test group can be done over time as a follow-up.
The important thing is to quickly place devs in a position to easily
check that fixes have no unintended side-effects. Very nice!
regards,
Andrew Dinn
-----------
Senior Principal Software Engineer
Red Hat UK Ltd
Registered in England and Wales under Company Registration No. 03798903
Directors: Michael Cunningham, Michael ("Mike") O'Neill, Eric Shander
