RFR 8245194: Unix domain socket channel implementation
Alan Bateman
Alan.Bateman at oracle.com
Thu Aug 13 08:58:53 UTC 2020
On 12/08/2020 13:01, Michael McMahon wrote:
> Thanks for the reviews of the API for this work (JEP 380). The CSR has
> been approved.
>
> So, I would like to continue with the implementation review. The full
> webrev can be found at:
>
> http://cr.openjdk.java.net/~michaelm/8245194/impl.webrev/webrev.8/
>
> Comments welcome.!
This is great work but I do have quite a lot of comments.
One thing that to start with is Net.unixDomainConnect/unixDomainBind are
taking String objects. The encoded bytes are already in UnxPath. Also
WindowsPath has the file path to specify to Win32 calls. So I think this
has to change to use getByteArrayForSysCalls and getPathForWin32Calls,
and this will mean refactoring because it is platform specific. It will
mean that the native code is simpler as it will be passed a pointer for
sun_path.
-Alan
More information about the nio-dev
mailing list