On 01/05/2020 16:53, Patrick Concannon wrote:
Hi,
I've refactored the test to use the name 'DatagramSocketSupplier'
instead of 'DSF' for the @FunctionalInterface, and used the supplier
method as Daniel suggested.
The updates can be found in the patch below.
http://cr.openjdk.java.net/~pconcannon/8243488/webrevs/webrev.03/
The tests looks good. Trivially you might to be consider renaming the
"factory" to "supplier" as it's a bit strange to have a
DatagramSocketSupplier named "factory". No need for a new webrev if you
take up that suggestion.
-Alan.