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

[Neil Richards]

Both now pushed to jdk7u-dev/jdk [1] [2].

Regards,
Neil

[1] http://hg.openjdk.java.net/jdk7u/jdk7u-dev/jdk/rev/e26bec65c55f
[2] http://hg.openjdk.java.net/jdk7u/jdk7u-dev/jdk/rev/cb92fa2fc48e


On Mon, 2012-11-05 at 06:09 +0100, Edvard Wendelin wrote:
> 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
> 


-- 
Unless stated above:
IBM email: neil_richards at uk.ibm.com
IBM United Kingdom Limited - Registered in England and Wales with number 741598.
Registered office: PO Box 41, North Harbour, Portsmouth, Hampshire PO6 3AU