RFR: 8273401: Remove JarIndex support in URLClassPath [v2]

wxiang github.com+53162078+shiyuexw at openjdk.java.net
Wed Sep 8 06:22:49 UTC 2021


On Tue, 7 Sep 2021 07:15:45 GMT, Alan Bateman <alanb at openjdk.org> wrote:

>> wxiang has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Some minor changes
>>   
>>   Include:
>>   1. remove public for INDEX_NAME in JarFile
>>   2. recover the definition for INDEX_NAME in JarIndex
>>   3. mv the test case JarIndexMergeTest.java into test/jdk/java/util/jar
>
> test/jdk/sun/misc/JarIndex/JarIndexMergeTest.java line 44:
> 
>> 42: import sun.tools.jar.JarIndex;
>> 43: 
>> 44: public class JarIndexMergeTest {
> 
> Is this the one remaining test in sun/misc/JarIndex? I think it can be moved to test/jdk/java/util/jar. I think we should change the package name in the summary comment too.

This is the only test case, and I move it to test/jdk/java/util/jar.
The summary comment is also changed.

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

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



More information about the security-dev mailing list