<html>
  <head>
    <meta http-equiv="Content-Type" content="text/html; charset=utf-8">
  </head>
  <body text="#000000" bgcolor="#FFFFFF">
    Hi Joe,<br>
    <br>
    A suggestion for more compact code: the (ever growing) switch
    statement in <span class="new">enum SourceTarget.checksrc could be
      replaced with a field of type BiConsumer<Versions,
      String[]>, initialized in constructor. For each enum constant
      you then just pass a method reference to the chosen target method.
      The checksrc in </span><span class="new"><span class="new">SourceTarget
        could be an instance method.<br>
        <br>
      </span>Peter<br>
      <br>
    </span>
    <div class="moz-cite-prefix">On 06/29/2018 09:41 PM, joe darcy
      wrote:<br>
    </div>
    <blockquote type="cite"
      cite="mid:7c73f7f9-ab06-f734-633a-35173a2d45a9@oracle.com">Dis-charged
      revision along with JDK 11 specific source example now up at
      <br>
      <br>
          <a class="moz-txt-link-freetext" href="http://cr.openjdk.java.net/~darcy/8206085.1/">http://cr.openjdk.java.net/~darcy/8206085.1/</a>
      <br>
      <br>
      Thanks,
      <br>
      <br>
      -Joe
      <br>
      <br>
      <br>
      On 6/29/2018 9:55 AM, Jonathan Gibbons wrote:
      <br>
      <blockquote type="cite">There's more static in this file than on
        my high-school Van de Graaf generator!
        <br>
        <br>
        Can we follow the convention of creating an instance in main,
        and then using instance methods.
        <br>
        <br>
        The naming of some methods is also novel, with respect to
        case-conventions.
        <br>
        <br>
        -- Jon
        <br>
        <br>
        <br>
        On 6/28/18 8:15 PM, joe darcy wrote:
        <br>
        <blockquote type="cite">Hello,
          <br>
          <br>
          Fresh off of updating
          <br>
          <br>
              langtools/tools/javac/versions/Versions.java
          <br>
          <br>
          for the JDK 11 -> 12 transition, I'd like to refactor the
          test to reduce the maintenance needed when adding new versions
          (as in new releases) or removing versions (as is planned for
          later in JDK 12):
          <br>
          <br>
              <a class="moz-txt-link-freetext" href="http://cr.openjdk.java.net/~darcy/8206085.0/">http://cr.openjdk.java.net/~darcy/8206085.0/</a>
          <br>
          <br>
          Thanks,
          <br>
          <br>
          -Joe
          <br>
          <br>
        </blockquote>
        <br>
      </blockquote>
      <br>
    </blockquote>
    <br>
  </body>
</html>