RFR: 8157716: jdk.internal.loader.ClassLoaders.addURLToUCP() should return converted real path URL

[Martin Buchholz]

Ohh, I missed that calling toRealPath was existing behavior, not
changed by webrev.00
I agree changing that sort of thing is risky.

Will toFileURL really be called by the VM?  It's not now.  I'm
surprised the VM would want an actual URL instead of a canonicalized
native file name.



On Wed, May 25, 2016 at 11:43 AM, Alan Bateman <Alan.Bateman at oracle.com> wrote:
>
>
> On 25/05/2016 19:28, Jiangli Zhou wrote:
>>
>> Here is the updated webrev:
>>
>>    http://cr.openjdk.java.net/~jiangli/8157716/webrev.01/
>
> This patch changes long standing behavior in URLs to resources on the class
> path will no longer be URLs to the canonical file path. It's might be okay
> but it's impossible to know. I thought the first patch was okay, except for
> the typo that Martin pointed out.
>
> -Alan