Looks good.

/Erik

On 2015-06-10 15:44, Magnus Ihse Bursie wrote:
On 2015-06-09 01:31, Daniel D. Daugherty wrote:
> http://cr.openjdk.java.net/~ihse/JDK-8085822-JEP-223-initial-patch/webrev.01

General comment: Not all copyright years were updated.

I realize I missed that part of the review. :-(

I have now updated the copyright years. Here's a delta review:

http://cr.openjdk.java.net/~ihse/JDK-8085822-JEP-223-initial-patch-delta-02/webrev.01/

Here's the complete review once again:

http://cr.openjdk.java.net/~ihse/JDK-8085822-JEP-223-initial-patch/webrev.03

Hopefully this is now the final version to be pushed to verona, and that any additional problems can be resolved with follow-up patches.

/Magnus

Reply via email to