stan/jdk-24
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

8139820: URLClassPath.FileLoader constructor redundantly checks protocol

Browse Source 
Reviewed-by: chegar, redestad
This commit is contained in:
Evgeny Mandrikov 2019-08-21 00:08:35 +02:00
parent f83ac54e23
commit 444e784ce1
1 changed files with 8 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

16
src/java.base/share/classes/jdk/internal/loader/URLClassPath.java
View File

@ -699,7 +699,7 @@ public class URLClassPath {
/*
* Nested class used to represent a Loader of resources from a JAR URL.
*/
static class JarLoader extends Loader {
private static class JarLoader extends Loader {
private JarFile jar;
private final URL csu;
private JarIndex index;
@ -714,9 +714,9 @@ public class URLClassPath {
* Creates a new JarLoader for the specified URL referring to
* a JAR file.
*/
JarLoader(URL url, URLStreamHandler jarHandler,
HashMap<String, Loader> loaderMap,
AccessControlContext acc)
private JarLoader(URL url, URLStreamHandler jarHandler,
HashMap<String, Loader> loaderMap,
AccessControlContext acc)
throws IOException
{
super(new URL("jar", "", -1, url + "!/", jarHandler));
@ -1193,11 +1193,11 @@ public class URLClassPath {
/* Canonicalized File */
private File dir;
FileLoader(URL url) throws IOException {
/*
* Creates a new FileLoader for the specified URL with a file protocol.
*/
private FileLoader(URL url) throws IOException {
super(url);
if (!"file".equals(url.getProtocol())) {
throw new IllegalArgumentException("url");
}
String path = url.getFile().replace('/', File.separatorChar);
path = ParseUtil.decode(path);
dir = (new File(path)).getCanonicalFile();