<html>
<head>
<meta content="text/html; charset=ISO-8859-1"
http-equiv="Content-Type">
</head>
<body text="#000000" bgcolor="#FFFFFF">
<div class="moz-cite-prefix">Hi, Kelly.<br>
Looks like changes in these files are unnecessary.<br>
<p><code>
<a
href="http://cr.openjdk.java.net/%7Eohair/openjdk8/generateHeaders1/webrev/src/macosx/classes/sun/lwawt/LWLabelPeer.java.cdiff.html">Cdiffs</a>
<a
href="http://cr.openjdk.java.net/%7Eohair/openjdk8/generateHeaders1/webrev/src/macosx/classes/sun/lwawt/LWLabelPeer.java.udiff.html">Udiffs</a>
<a
href="http://cr.openjdk.java.net/%7Eohair/openjdk8/generateHeaders1/webrev/src/macosx/classes/sun/lwawt/LWLabelPeer.java.sdiff.html">Sdiffs</a>
<a
href="http://cr.openjdk.java.net/%7Eohair/openjdk8/generateHeaders1/webrev/src/macosx/classes/sun/lwawt/LWLabelPeer.java.frames.html">Frames</a>
<a
href="http://cr.openjdk.java.net/%7Eohair/openjdk8/generateHeaders1/webrev/src/macosx/classes/sun/lwawt/LWLabelPeer.java-.html">Old</a>
<a
href="http://cr.openjdk.java.net/%7Eohair/openjdk8/generateHeaders1/webrev/src/macosx/classes/sun/lwawt/LWLabelPeer.java.html">New</a>
<a
href="http://cr.openjdk.java.net/%7Eohair/openjdk8/generateHeaders1/webrev/src/macosx/classes/sun/lwawt/LWLabelPeer.java.patch">Patch</a>
<a
href="http://cr.openjdk.java.net/%7Eohair/openjdk8/generateHeaders1/webrev/raw_files/new/src/macosx/classes/sun/lwawt/LWLabelPeer.java">Raw</a>
</code>
<b>src/macosx/classes/sun/lwawt/LWLabelPeer.java</b> </p>
<blockquote>
<span class="lineschanged">4 lines changed: 4 ins; 0 del; 0 mod;
115 unchg</span>
</blockquote>
<p><code>
<a
href="http://cr.openjdk.java.net/%7Eohair/openjdk8/generateHeaders1/webrev/src/macosx/classes/sun/lwawt/LWTextFieldPeer.java.cdiff.html">Cdiffs</a>
<a
href="http://cr.openjdk.java.net/%7Eohair/openjdk8/generateHeaders1/webrev/src/macosx/classes/sun/lwawt/LWTextFieldPeer.java.udiff.html">Udiffs</a>
<a
href="http://cr.openjdk.java.net/%7Eohair/openjdk8/generateHeaders1/webrev/src/macosx/classes/sun/lwawt/LWTextFieldPeer.java.sdiff.html">Sdiffs</a>
<a
href="http://cr.openjdk.java.net/%7Eohair/openjdk8/generateHeaders1/webrev/src/macosx/classes/sun/lwawt/LWTextFieldPeer.java.frames.html">Frames</a>
<a
href="http://cr.openjdk.java.net/%7Eohair/openjdk8/generateHeaders1/webrev/src/macosx/classes/sun/lwawt/LWTextFieldPeer.java-.html">Old</a>
<a
href="http://cr.openjdk.java.net/%7Eohair/openjdk8/generateHeaders1/webrev/src/macosx/classes/sun/lwawt/LWTextFieldPeer.java.html">New</a>
<a
href="http://cr.openjdk.java.net/%7Eohair/openjdk8/generateHeaders1/webrev/src/macosx/classes/sun/lwawt/LWTextFieldPeer.java.patch">Patch</a>
<a
href="http://cr.openjdk.java.net/%7Eohair/openjdk8/generateHeaders1/webrev/raw_files/new/src/macosx/classes/sun/lwawt/LWTextFieldPeer.java">Raw</a>
</code>
<b>src/macosx/classes/sun/lwawt/LWTextFieldPeer.java</b> </p>
<blockquote>
<span class="lineschanged">4 lines changed: 4 ins; 0 del; 0 mod;
134 unchg</span>
</blockquote>
<br>
23.05.2012 05:14, Kelly O'Hair wrote:<br>
</div>
<blockquote
cite="mid:A15AFA68-B8B8-4549-A55A-8A89F9055C98@oracle.com"
type="cite">
<pre wrap="">
7170969: Add @GenerateNativeHeader to classes whose fields need to be exported for JNI
<a class="moz-txt-link-freetext" href="http://cr.openjdk.java.net/~ohair/openjdk8/generateHeaders1/webrev/">http://cr.openjdk.java.net/~ohair/openjdk8/generateHeaders1/webrev/</a>
The new build infrastructure wants to fully automate the generation of the JNI header files (running javah).
On classes that have "native" methods this is obvious, but class that only export constants, and whose constants
are needed in the JNI native code, we need to mark these classes as needing their native header file generated.
This change is adding:
import javax.tools.annotation.GenerateNativeHeader;
/* No native methods here, but the constants are needed in the supporting JNI code */
@GenerateNativeHeader
To any class without a native method, but needing to expose it's constants to the native code.
-kto
</pre>
</blockquote>
<br>
<br>
<pre class="moz-signature" cols="72">--
Best regards, Sergey. </pre>
</body>
</html>