New constructor in sun.tools.java.ClassPath builds a path using File.separator instead of File.pathSeparatorChar (original) (raw)

Deven You youdwei at linux.vnet.ibm.com
Thu Apr 12 07:22:01 UTC 2012

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

More information about the core-libs-dev mailing list