adding rsockets support into JDK
yingqi.lu at intel.com
Thu Oct 4 22:28:50 UTC 2018
Please find version 14 of the patch available at http://cr.openjdk.java.net/~ylu/8195160.14/
In this version, I use shared secrets mechanism to avoid changing two setXXX methods from “package private” to “protected” in SocketImpl. Please review and let me know if this is a reasonable change.
I also modified CSR accordingly at https://bugs.openjdk.java.net/browse/JDK-8205186.
From: Brian Burkhalter [mailto:brian.burkhalter at oracle.com]
Sent: Wednesday, October 3, 2018 5:18 PM
To: Lu, Yingqi <yingqi.lu at intel.com>
Cc: Aundhe, Shirish <shirish.aundhe at intel.com>; nio-dev at openjdk.java.net; Viswanathan, Sandhya <sandhya.viswanathan at intel.com>; Kaczmarek, Eric <eric.kaczmarek at intel.com>
Subject: Re: adding rsockets support into JDK
On Oct 3, 2018, at 5:02 PM, Lu, Yingqi <yingqi.lu at intel.com<mailto:yingqi.lu at intel.com>> wrote:
Thank you for all the help here!
I also modified the CSR athttps://bugs.openjdk.java.net/browse/JDK-8205186 by adding following lines to the end of it. I think we need to include all the API changes here?
Yes, all visible API changes need to be included.
In addition, we will change the access modifier on the following methods:
java.net.ServerSocket: ServerSocket(SocketImpl impl) from package private to protected
java.net.SocketImpl: setSocket(Socket soc) and setServerSocket(ServerSocket soc) from package private to protected
On these latter two items, I recall that Alan had an objection:
"On the API changes then I see that two protected setXXXX methods have been added to SocketImpl. I think we should try to avoid that, even if it means using the shared secrets mechanism.” 
Please review and let me know if there is anything missing.
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the nio-dev