Thanks for reviewing!

For the last modified part, we can discuss updating the header script offline. Many of the classes have already used this format, I hope you're okay with me checking in this changeset.

Thanks,
Joe

On 11/28/2017 10:19 AM, joe darcy wrote:
Hi Joe,

The code changes look fine, but the copyright blocks should *not* be updated to include a "@LastModified: Nov 2017" comment.

Cheers,

-Joe


On 11/28/2017 10:11 AM, Joe Wang wrote:
Hi,

Please review a fix for a few more deprecation warnings. Compiling with -Xlint:all showed that these were the last few warnings. We can then enable -Xlint:all for the java.xml module.

JBS: https://bugs.openjdk.java.net/browse/JDK-8191938
webrevs: http://cr.openjdk.java.net/~joehw/jdk10/8191938/webrev/

Thanks,
Joe


Reply via email to