RFR: 8345431: Improve jar --validate to detect duplicate or invalid entries [v12]

Jaikiran Pai jpai at openjdk.org
Thu May 22 12:07:54 UTC 2025


On Thu, 22 May 2025 01:29:12 GMT, Henry Jen <henryjen at openjdk.org> wrote:

>> This PR check the jar file to ensure entries are consistent from the central directory and local file header. Also check there is no duplicate entry names that could override the desired content by accident.
>
> Henry Jen has updated the pull request incrementally with one additional commit since the last revision:
> 
>   Validator to take a file, use term JAR file

Hello Henry, thank you for this enhancement. Overall this looks very good to me, especially because the validation code is concise and very cleanly done. I only had some nits and minor clarifications in this review.

Some parts of the test (like the usage of ISO_8859_1 charset) isn't clear to me, so I'll come back to that later. I'll focus on the CSR review now.

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

PR Comment: https://git.openjdk.org/jdk/pull/24430#issuecomment-2900987846


More information about the core-libs-dev mailing list