On 07/05/2019, 17:14, Alan Bateman wrote:
On 07/05/2019 16:44, Michael McMahon wrote:
Hi Alan,
What's the purpose of the change to the UdpTest?
The deprecated Socket constructors can be used to create a Socket that
uses UDP. This means the new SocketImpl has to support the creation of
UDP sockets and use the ResourceManager to enforce limits when running
with a security manager. So it's just increasing test coverage as I
wasn't able to find any tests in test/jdk that exercise it.
-Alan
Ok, thanks. A test that might be useful could be one that compares the
supported options between the old
and new impls for both client and server sockets. I guess it shouldn't
be required for the initial push though.
- Michael.
