Looks good Alan!

best regards.

-- daniel

On 14/01/20 19:23, Alan Bateman wrote:
Thanks for going through this. I tried to keep the changes to DatagramSocket to a minimum but I don't mind making an exception for oldImpl. It's a slippery slope as there is a lot of technical debt in this area. Also oldImpl is the the support for DatagramSocketImpl that were compiled for JDK 1.3 or older and I really hope we can drop that code soon.

Here's the updated webrev that changes oldImpl to be final.
    http://cr.openjdk.java.net/~alanb/8236925/2/webrev/

-Alan

Reply via email to