Request for approval: 6610897: New constructor in sun.tools.java.ClassPath builds a path using File.separator instead of File.pathSeparator

Alan Bateman Alan.Bateman at oracle.com
Fri Nov 2 04:31:08 PDT 2012


On 02/11/2012 02:55, Deven You wrote:
> Hi Alan,
>
> Thanks very much for your correction. I have updated the webre[1] 
> according to your comments. Please review it.
>
> [1] http://cr.openjdk.java.net/~youdwei/ojdk-505/7u/webrev.02/ 
> <http://cr.openjdk.java.net/%7Eyoudwei/ojdk-505/7u/webrev.02/>
It looks okay to me but it requires one of the jdk7u maintainers to 
approve. Also one of things that we've been trying to do is keep the 
jdk7u change-sets the same as what is in jdk8 so I think it would be 
better to hg import both both change-sets rather than combining them 
into one.

-Alan



More information about the jdk7u-dev mailing list