<core-libs-dev> New constructor in sun.tools.java.ClassPath builds a path using File.separator instead of File.pathSeparatorChar

Charles Lee littlee at linux.vnet.ibm.com
Fri Apr 13 02:21:12 UTC 2012


On 04/12/2012 04:26 PM, Alan Bateman wrote:
> On 12/04/2012 08:22, Deven You wrote:
>>
>> I have verified this commit. Thanks Alan and Charles!
> The change-set that Charles pushed added test/sun/tools/classpath/ 
> RMICClassPathTest.java but I didn't see it in the original webrev. I 
> see now that your original mail had the proposed test in a separate 
> webrev but I missed that. Anyway, one comment is that test has been 
> added to an odd location that doesn't correspond to where the fix was 
> made. I think test/sun/rmi/rmic would have been a more suitable location.
>
> -Alan.
Hi Alan,

I am sorry for that. Should I commit a revert changeset?

-- 
Yours Charles




More information about the core-libs-dev mailing list