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

Edvard Wendelin edvard.wendelin at oracle.com
Sun Nov 4 21:09:00 PST 2012


Hi, 

Approved, but please maintain the two changesets as  Alan described.

Cheers,
Edvard

On Nov 2, 2012, at 12:31 PM, Alan Bateman <Alan.Bateman at oracle.com> wrote:

> 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