<html>
  <head>
    <meta content="text/html; charset=utf-8" http-equiv="Content-Type">
  </head>
  <body bgcolor="#FFFFFF" text="#000000">
    Steve,<br>
    <br>
    The most noticeable ickiness is the following string bashing on the
    name of the Target enum constant, <br>
    <pre><span class="new"> 249             List<String> list = List.of(target.toString().replace("JDK1_", ""));</span></pre>
    It's not wrong, but it is fragile, as we've already considered
    renaming Target.JDK1_9 to Target.JDK9.<br>
    <br>
    A somewhat more robust solution would be to use a regex [1]:<br>
    <pre><span class="new"> 249             List<String> list = List.of(target.toString().replaceAll("^JDK(1_)?", ""));</span></pre>
    The gold-star solution would be to have a method on Target to return
    the MR-version number.<br>
    <br>
    -- Jon<br>
    <br>
    [1]
<a class="moz-txt-link-freetext" href="http://blog.codinghorror.com/regular-expressions-now-you-have-two-problems/">http://blog.codinghorror.com/regular-expressions-now-you-have-two-problems/</a><br>
    <br>
    <br>
    <div class="moz-cite-prefix">On 04/05/2016 02:02 PM, Steve Drach
      wrote:<br>
    </div>
    <blockquote
      cite="mid:ABE754D4-F784-4B85-974E-71C96BB01826@oracle.com"
      type="cite">
      <meta http-equiv="Content-Type" content="text/html; charset=utf-8">
      <meta http-equiv="Content-Type" content="text/html; charset=utf-8">
      Hi,
      <div class=""><br class="">
      </div>
      <div class="">Please review the changes required to make javac and
        the StandardJavaFileManager multi-release jar aware.  For javac,
        the version of the classes in a multi-release.jar is selected by
        the -release (or -target) command line option, or if the option
        is not present, javac’s default policy is used to select the
        version — typically the runtime version (i.e. version 9 is
        selected for JDK 9).  For the StandardJavaFileManager, the
        version is selected with the handleOption method.  See the tests
        for more detail.</div>
      <div class=""><br class="">
      </div>
      <div class="">issue: <a moz-do-not-send="true"
          href="https://bugs.openjdk.java.net/browse/JDK-8149757"
          class="">https://bugs.openjdk.java.net/browse/JDK-8149757</a></div>
      <div class="">webrev: <a moz-do-not-send="true"
          href="http://cr.openjdk.java.net/%7Esdrach/8149757/webrev/index.html"
          class="">http://cr.openjdk.java.net/~sdrach/8149757/webrev/index.html</a></div>
      <div class=""><br class="">
      </div>
      <div class="">Thank you,</div>
      <div class="">Steve</div>
    </blockquote>
    <br>
  </body>
</html>