On Wed, 26 Mar 2025 12:48:25 GMT, Sean Mullan <mul...@openjdk.org> wrote:
>> Hai-May Chao has updated the pull request incrementally with one additional >> commit since the last revision: >> >> Updated with Sean's comments > > src/jdk.jartool/share/classes/sun/security/tools/jarsigner/resources/jarsigner.properties > line 216: > >> 214: >> manifest.attribute.1.differs.jarfile.value.2.jarinputstream.value.3=Manifest >> attribute %1$s differs: JarFile value = %2$s, JarInputStream value = %3$s >> 215: >> Entry.1.present.when.reading.jarinputstream.but.missing.via.JarFile=Entry %s >> is present when reading via JarInputStream but missing when reading via >> JarFile >> 216: entry.1.present.in.jarfile.but.unreadable=Entry %s is present in >> JarFile but unreadable > > Shouldn't this be the opposite of > "entry.1.present.when.reading.jarinputstream.but.missing.via.JarFile": > > "Entry %s is present when reading via JarFile but missing when reading via > JarInputStream" Actually never mind, I see that this warning is if there is an entry but no input stream. So ignore my comment. ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/23532#discussion_r2014355667