On Mon, 23 Sep 2024 05:11:34 GMT, Jaikiran Pai <j...@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. This pull request has now been integrated. Changeset: 865d99f6 Author: Jaikiran Pai <j...@openjdk.org> URL: https://git.openjdk.org/jdk/commit/865d99f63475799b9a0503a3dcc21a7534b014d1 Stats: 5 lines in 1 file changed: 0 ins; 4 del; 1 mod 8340596: Remove dead code from RequiresSetenv function in java.base/unix/native/libjli/java_md.c Reviewed-by: dholmes ------------- PR: https://git.openjdk.org/jdk/pull/21125
