
<html>

  <head>

    <meta http-equiv="Content-Type" content="text/html; charset=UTF-8">

  </head>

  <body>

    On 01/05/2020 16:53, Patrick Concannon wrote:<br>

    <blockquote type="cite"

      cite="mid:05170ef4-8cbf-2c89-7546-038d4ed1b283@oracle.com">

      <meta http-equiv="Content-Type" content="text/html; charset=UTF-8">

      <p><tt>Hi,</tt></p>

      <p><tt>I've refactored the test to use the name '</tt><tt>DatagramSocketSupplier'

          instead of 'DSF' for the @FunctionalInterface, and used the

          supplier method as Daniel suggested. <br>

          <br>

          The updates can be found in the patch below.</tt><a

          moz-do-not-send="true"

          href="http://cr.openjdk.java.net/~pconcannon/8243488/webrevs/webrev.03/"><tt><br>

            <br>

http://cr.openjdk.java.net/~pconcannon/8243488/webrevs/webrev.03/</tt></a></p>

    </blockquote>

    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.<br>

    <br>

    -Alan.<br>

  </body>

</html>