Can I please get a review of this change which cleans up the `libzip` ZIP utility library in the JDK?
As noted in https://bugs.openjdk.org/browse/JDK-8390647, over time, this library has accumulated some functions which are no longer used. The commit in this PR removes those unused functions and moves some internal functions in to the implementation (`.c` file) instead of having it in the header file. Some macros too have been moved into the implementation. No new test has been introduced but existing tests in tier1, tier2 and tier3 continue to pass with this change. Once the PR is reviewed, I'll run higher tiers too before integrating. --------- - [x] I confirm that I make this contribution in accordance with the [OpenJDK Interim AI Policy](https://openjdk.org/legal/ai). ------------- Commit messages: - move macros out of header file - move some functions as internal details Changes: https://git.openjdk.org/jdk/pull/32448/files Webrev: https://webrevs.openjdk.org/?repo=jdk&pr=32448&range=00 Issue: https://bugs.openjdk.org/browse/JDK-8390647 Stats: 341 lines in 2 files changed: 145 ins; 183 del; 13 mod Patch: https://git.openjdk.org/jdk/pull/32448.diff Fetch: git fetch https://git.openjdk.org/jdk.git pull/32448/head:pull/32448 PR: https://git.openjdk.org/jdk/pull/32448
