RFR: 8340596: Remove dead code from RequiresSetenv function in java.base/unix/native/libjli/java_md.c

David Holmes dholmes at openjdk.org
Mon Sep 23 06:38:35 UTC 2024


On Mon, 23 Sep 2024 05:11:34 GMT, Jaikiran Pai <jpai at openjdk.org> wrote:

> Can I please get a review of this trivial change which removes dead code from the `RequiresSetenv()` function?
> 
> As noted in https://bugs.openjdk.org/browse/JDK-8340596 this appears to be a left-over from the changes in https://bugs.openjdk.org/browse/JDK-8244224.
> 
> No new tests have been added and existing tier1, tier2 and tier3 tests continue to pass.

Looks good and trivial.

Thanks

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

Marked as reviewed by dholmes (Reviewer).

PR Review: https://git.openjdk.org/jdk/pull/21125#pullrequestreview-2321225955


More information about the core-libs-dev mailing list