Thanks for pointing out where the review was, I don't check the update
email list as often.
Reviewed in jdk8u-dev. Looks good.
Brad
On 3/6/2018 1:03 AM, Seán Coffey wrote:
Thanks Brad - yes, this review is for JDK 11 (jdk/jdk).
I've another review request open for you in order to handle the
jdk8u-dev edit :
http://mail.openjdk.java.net/pipermail/jdk8u-dev/2018-March/007284.html
regards,
Sean.
On 05/03/2018 23:41, Brad Wetmore wrote:
Looks good.
This looks like it still needs to be done in jdk/jdk. Are you
planning on doing that?
Thanks,
Brad
On 3/5/2018 6:37 AM, Seán Coffey wrote:
Brad pointed out to me off-line that the test case from 8193892 needs
correcting. Some late editing to the test broke functionality. I've
corrected the testcase and added extra checks to ensure that
"MyProvider" is being registered correctly.
https://bugs.openjdk.java.net/browse/JDK-8199018
http://cr.openjdk.java.net/~coffeys/webrev.8199018/webrev/