RFR: 8344078: Remove security manager dependency in java.nio [v4]

Alan Bateman alanb at openjdk.org
Mon Nov 18 16:43:57 UTC 2024


On Mon, 18 Nov 2024 16:32:14 GMT, Brian Burkhalter <bpb at openjdk.org> wrote:

>> Expunge the use of the `SecurityManager`, `AccessController`, and the like from the `java.nio` and `sun.nio` package hierarchies.
>
> Brian Burkhalter has updated the pull request incrementally with one additional commit since the last revision:
> 
>   8344078: Address reviewer comments since previous commit except about InnoucuousThread

Good work, good riddance, good cleanup.

src/java.base/share/classes/sun/nio/ch/ServerSocketChannelImpl.java line 732:

> 730:                     sb.append("unbound");
> 731:                 } else if (isUnixSocket()) {
> 732:                     sb.append(addr.toString());

Here's another case where the toString isn't needed.

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

Marked as reviewed by alanb (Reviewer).

PR Review: https://git.openjdk.org/jdk/pull/22132#pullrequestreview-2443162053
PR Review Comment: https://git.openjdk.org/jdk/pull/22132#discussion_r1846916300


More information about the nio-dev mailing list