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

Brian Burkhalter bpb at openjdk.org
Mon Nov 18 17:04:10 UTC 2024


On Mon, 18 Nov 2024 16:38:12 GMT, Alan Bateman <alanb at openjdk.org> wrote:

>> 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
>
> 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.

Fixed in 7b7f23c. Prior resolved conversations addressed in bc563ae.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/22132#discussion_r1846948666


More information about the nio-dev mailing list