Re: Code review for 6915797 7090178

2011-09-14 Thread Alan Bateman
Mandy Chung wrote: 6915797: Remove sun.tools.jar.JarImageSource that is not used 7090178: Move java.util.XMLUtils to another package to avoid split package Webrev at: http://cr.openjdk.java.net/~mchung/6915797/webrev.00/ The synopsis says it all. Mostly looks okay to me too. A few

Code review for 6915797 7090178

2011-09-13 Thread Mandy Chung
6915797: Remove sun.tools.jar.JarImageSource that is not used 7090178: Move java.util.XMLUtils to another package to avoid split package Webrev at: http://cr.openjdk.java.net/~mchung/6915797/webrev.00/ The synopsis says it all. Thanks Mandy

Re: Code review for 6915797 7090178

2011-09-13 Thread Xueming Shen
Looks fine. On 9/13/2011 1:54 PM, Mandy Chung wrote: 6915797: Remove sun.tools.jar.JarImageSource that is not used 7090178: Move java.util.XMLUtils to another package to avoid split package Webrev at: http://cr.openjdk.java.net/~mchung/6915797/webrev.00/ The synopsis says it all.