RFR: 8377910: Minor cleanup of java/io/FileDescriptor/Sharing.java

Brian Burkhalter bpb at openjdk.org
Fri Feb 13 17:21:32 UTC 2026


On Fri, 13 Feb 2026 17:14:31 GMT, Brian Burkhalter <bpb at openjdk.org> wrote:

> Null checks potentially mask a more serious, fundamental problem and should be removed to allow a `NullPointerException` to be throw if the variable is `null`.

It appears that the `null` checks in question were intended to match others elsewhere in the test where the checks are in a finally block and there are multiple variables among which not all might be `null` depending on which variables were initialized in the `try` block.

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

PR Comment: https://git.openjdk.org/jdk/pull/29718#issuecomment-3898346004


More information about the core-libs-dev mailing list