On Fri, 10 Mar 2023 07:47:05 GMT, Jaikiran Pai <j...@openjdk.org> wrote:
>> Eirik Bjorsnos has updated the pull request incrementally with one >> additional commit since the last revision: >> >> Remove reference to python in the @summary of >> DataDescriptorSignatureMissing > > test/jdk/java/util/zip/DataDescriptorSignatureMissing.java line 29: > >> 27: * @summary Check ability to read zip files created by python zipfile >> 28: * implementation, which fails to write optional (but recommended) data >> 29: * descriptor signatures. > > Hello Eirik, I think the summary should no longer have references to python. > Since this test was introduced for > https://bugs.openjdk.org/browse/JDK-8056934, perhaps we can just change the > `@summary` of this test definition to say: >> >>@summary Verify the ability to read zip files whose local header data >>descriptor is missing the optional signature Thanks, I updated the summary according to your suggestion. I left it there to keep the history/context of the test, but I guess that's a bit detailed and is already captured in the bug. ------------- PR: https://git.openjdk.org/jdk/pull/12959