RFR: 8345431: Detect duplicate entries in jar files with jar --validate [v6]

Henry Jen henryjen at openjdk.org
Thu May 15 22:03:52 UTC 2025


On Wed, 7 May 2025 18:00:23 GMT, Henry Jen <henryjen at openjdk.org> wrote:

>> src/jdk.jartool/share/classes/sun/tools/jar/Validator.java line 127:
>> 
>>> 125: 
>>> 126:         public DedupEntryNames(boolean isCEN) {
>>> 127:             this.isCEN = isCEN;
>> 
>> Might be clearer if this was an enum vs boolean?
>
> I feel the same way but wondering if that's an overkill.

Refactored to consolidate validation on entries into a standalone EntryValidator. Hope that's easier to understand.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/24430#discussion_r2092018706


More information about the core-libs-dev mailing list