> Remove ineffective/unused ImageBufferCache class, and simplify callers / > remove dead code. > > I removed the release methods in the internal classes, but the public > ModuleReader API method is still there (the override can go away though since > the default implementation also tests for non-null, so removing the override > has no risk). > > I suspect there are no implementations of ModuleReader that implement release > semantics after this change, so perhaps we could relax the documentation > around it? Thoughts welcome.
David Beaumont has updated the pull request incrementally with one additional commit since the last revision: copyright update ------------- Changes: - all: https://git.openjdk.org/jdk/pull/29043/files - new: https://git.openjdk.org/jdk/pull/29043/files/5a37317e..e3bbc4c5 Webrevs: - full: https://webrevs.openjdk.org/?repo=jdk&pr=29043&range=02 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=29043&range=01-02 Stats: 3 lines in 3 files changed: 0 ins; 0 del; 3 mod Patch: https://git.openjdk.org/jdk/pull/29043.diff Fetch: git fetch https://git.openjdk.org/jdk.git pull/29043/head:pull/29043 PR: https://git.openjdk.org/jdk/pull/29043
