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.

Reply via email to