RFR: 8245194: Unix domain socket channel implementation [v18]
Michael McMahon
michaelm at openjdk.java.net
Wed Oct 14 13:21:19 UTC 2020
On Tue, 13 Oct 2020 15:50:57 GMT, Daniel Fuchs <dfuchs at openjdk.org> wrote:
>> Michael McMahon has updated the pull request incrementally with one additional commit since the last revision:
>>
>> unixdomainchannels: updates from Chris's review 9 Oct 2020
>
> src/java.base/share/classes/java/net/UnixDomainSocketAddress.java line 134:
>
>> 132: if (fs.getClass().getModule() != Object.class.getModule()) {
>> 133: throw new IllegalArgumentException();
>> 134: }
>
> I believe it would be better if this verification was done outside of the constructor (you could do it in the of(Path)
> static factory method) - this would avoid creating the object if the path is invalid.
Good point.
-------------
PR: https://git.openjdk.java.net/jdk/pull/52
More information about the hotspot-jfr-dev
mailing list