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