Thanks for pointing out where the review was, I don't check the update email list as often.

Reviewed in jdk8u-dev.  Looks good.


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 :


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?



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.

Reply via email to