RFR: 8273401: Remove JarIndex support in URLClassPath

Lance Andersen lancea at openjdk.java.net
Tue Sep 7 16:33:36 UTC 2021


On Tue, 7 Sep 2021 03:34:27 GMT, wxiang <github.com+53162078+shiyuexw at openjdk.org> wrote:

> There is a bug for URLClassPath.findResources with JarIndex.
> With some discussions about the bug, the current priority is to remove the JAR index support in URLClassPath, 
> and don’t need to do anything to the jar tool in the short term, except just to move JarIndex to the jdk.jartool module. 
> 
> The PR includes:
> 1. remove the JarIndex support in URLClassPath
> 2. move JarIndex into  jdk.jartool module.

I think this looks good overall.

Once you do your next set up updates, I will run it through mach5 for you

src/jdk.jartool/share/classes/sun/tools/jar/Main.java line 73:

> 71: import static java.util.jar.JarFile.MANIFEST_NAME;
> 72: import static java.util.stream.Collectors.joining;
> 73: import static java.util.jar.JarFile.INDEX_NAME;

Maybe move this above the java.util.streamCollectors.joining import

Also, the copyright should probably be updated to 2021

test/jdk/sun/misc/JarIndex/JarIndexMergeTest.java line 25:

> 23: 
> 24: /*
> 25:  * @test

Probably should update the copyright year to include 2021

-------------

PR: https://git.openjdk.java.net/jdk/pull/5383



More information about the security-dev mailing list