On 21/03/16 18:35, Alan Bateman wrote:
On 21/03/2016 17:45, Chris Hegarty wrote:
In the context of 8149122 & 8152190 it was noticed that
sun.rmi.registry.RegistryImpl is the only invoker of
URLClassPath::pathToURLs.
Rather that adding a qualified export to java.rmi for this, it makes
sense to move
pathToURLs(String) to RegistryImpI ( and "modernize" the
implementation a little ).
http://cr.openjdk.java.net/~chegar/8152277/
https://bugs.openjdk.java.net/browse/JDK-8152277
Rather than using ParseUtils then you could just do:
URL url = Paths.get(p).toRealPath().toUri().toURL();
This is a little more "modernization" than I originally signed
up for ;-) but it has the advantage of not further polluting
the qualified exports, which is good.
Also I assume that pathToURLs does not need to be public.
Right.
Update webrev:
http://cr.openjdk.java.net/~chegar/8152277/
I'm not sure under what circumstances toRealPath can fail, but
I tried to be consistent with the old behavior.
-Chris.
