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

[Deven You]

I have verified this commit. Thanks Alan and Charles!

On 04/12/2012 03:12 PM, Charles Lee wrote:
> On 04/11/2012 03:44 PM, Deven You wrote:
>> On 04/10/2012 04:12 PM, Alan Bateman wrote:
>>> On 09/04/2012 10:32, Deven You wrote:
>>>> :
>>>>
>>>> The issue is reported by one of our test team which maintains  a 
>>>> set of ORB test cases.
>>> Okay but I guess I'm still a bit curious as it's not obvious that 
>>> rmic (or rmic --iiop) invokes toString, maybe this is a white box 
>>> test or a test that is using sun.tools.java.*?
>>>
>>> -Alan.
>>>
>> Hi Alan,
>>
>> Though I am not very sure, I guess it is a white box test. I find 
>> some code snippet which may cause this errors, like:
>>
>>     sun.misc.URLClassPath.pathToURLs(classPath.toString()));
>>
>> I think if anyone use above like code, it will fail.
>>
>> Thanks a lot!
>>
> Hi Deven,
>
> The patch is committed @
>
> Changeset: c98a013ec628
> Author:    youdwei
> Date:      2012-04-12 15:04 +0800
> URL:http://hg.openjdk.java.net/jdk8/tl/jdk/rev/c98a013ec628
>
> 6610897: New constructor in sun.tools.java.ClassPath builds a path 
> using File.separator instead of File.pathSeparator
> Reviewed-by: alanb
>
> ! src/share/classes/sun/tools/java/ClassPath.java
> + test/sun/tools/classpath/RMICClassPathTest.java
>
>
>
> Please verify it.
>


-- 
Best Regards,

Deven