RFR: 8327208: Remove unused method java.util.jar.Manifest.make72Safe
Jaikiran Pai
jpai at openjdk.org
Tue Mar 5 04:28:44 UTC 2024
On Mon, 4 Mar 2024 09:55:09 GMT, Eirik Bjørsnøs <eirbjo at openjdk.org> wrote:
> Please review this cleanup PR which suggests we remove the package-private, unused and deprecated method `java.util.jar.Manifest.make72Safe`.
>
> This method was marked deprecated after a cleanup/refactoring in [JDK-8066619](https://bugs.openjdk.org/browse/JDK-8066619) caused it to fall out of use. It should rather be removed.
>
> Some tests reference the `make72Safe` methods in comment, or implement the legacy versions of the same logic to simulate pre JDK 11 line break behavior of `Manifest.write`. These comments and method names are adjusted to not reference the now removed method.
>
> This change was initially discussed here: https://mail.openjdk.org/pipermail/core-libs-dev/2024-February/119819.html
Hello Eirik, the removal of this internal unused package private method and the updated comments in the tests look good to me.
-------------
Marked as reviewed by jpai (Reviewer).
PR Review: https://git.openjdk.org/jdk/pull/18104#pullrequestreview-1915846886
More information about the core-libs-dev
mailing list