<html>
  <head>
    <meta http-equiv="Content-Type" content="text/html; charset=UTF-8">
  </head>
  <body text="#000000" bgcolor="#FFFFCC">
    Hi Claes,<br>
    <br>
    Looks fine.<br>
    <br>
    Thanks for the updates, Roger<br>
    <br>
    <br>
    <div class="moz-cite-prefix">On 7/11/19 10:39 AM, Claes Redestad
      wrote:<br>
    </div>
    <blockquote type="cite"
      cite="mid:2cbfd5d8-c4a0-4a2d-ba62-6fed14dc7155@oracle.com">Hi
      Roger,
      <br>
      <br>
      On 2019-07-11 16:10, Roger Riggs wrote:
      <br>
      <blockquote type="cite">Hi Claes,
        <br>
        <br>
        JavaLangAccess.java:
        <br>
        316: Add @param tag
        <br>
      </blockquote>
      <br>
      done.
      <br>
      <br>
      <blockquote type="cite">
        <br>
        System.java:  2282, 2287
        <br>
        Runtime.loadLibrary0 makes a second check for a security
        manager.
        <br>
        Is there any potential gain by calling ClassLoader.loadLibrary
        directly?
        <br>
        None of the internal uses would have a separatorChar.
        <br>
      </blockquote>
      <br>
      Most of the gain comes from not having to load one-off PA classes
      or
      <br>
      lambdas, but of course avoiding the indexOf and a few indirections
      are
      <br>
      marginally helpful to startup. We should at least assert that the
      <br>
      library names are sane, though.
      <br>
      <br>
      <blockquote type="cite">
        <br>
        I expect most of the files need a copyright update.
        <br>
        The script in <repo>/make/scripts/update_copyright_year.sh
        should do the work for modified files.
        <br>
      </blockquote>
      <br>
      Fixed copyrights and updated in place:
      <a class="moz-txt-link-freetext" href="http://cr.openjdk.java.net/~redestad/8227587/open.00">http://cr.openjdk.java.net/~redestad/8227587/open.00</a>
      <br>
      <br>
      Thanks!
      <br>
      <br>
      /Claes
      <br>
    </blockquote>
    <br>
  </body>
</html>