<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=utf-8">
</head>
<body text="#000000" bgcolor="#FFFFFF">
<pre><span class="removed">I see this opens was moved to platform-specific code ...
115 opens com.sun.java.swing.plaf.windows to</span>
<span class="removed"> 116 jdk.jconsole;</span></pre>
So jdk.jconsole definitely accesses this only on Windows ?<br>
<br>
-phil.<br>
<br>
<div class="moz-cite-prefix">On 11/28/2017 09:28 AM, Erik Joelsson
wrote:<br>
</div>
<blockquote type="cite"
cite="mid:171f8e59-8f7d-2ae4-a726-06e706297db3@oracle.com">From a
build point of view, this looks great.
<br>
<br>
/Erik
<br>
<br>
<br>
On 2017-11-27 18:27, Sergey Bylokhov wrote:
<br>
<blockquote type="cite">Hello.
<br>
Please review the fix for jdk10. This is the second attempt to
move windows L&F from non-windows platforms. The first
attempt JDK-6461834[1] was reverted because of JDK-8184813[2].
<br>
The root cause of those issue was fixed in JDK-8075255[3], and
now we can move it again.
<br>
<br>
Bug: <a class="moz-txt-link-freetext" href="https://bugs.openjdk.java.net/browse/JDK-8189656">https://bugs.openjdk.java.net/browse/JDK-8189656</a>
<br>
Webrev can be found at:
<a class="moz-txt-link-freetext" href="http://cr.openjdk.java.net/~serb/8189656/webrev.02">http://cr.openjdk.java.net/~serb/8189656/webrev.02</a>
<br>
<br>
The fix contains a few parts:
<br>
- The code related to win l&f was moved from the "share" to
the "windows" folder
<br>
- The platform-specific export was moved from module-info.java
to module-info.java.extra
<br>
- A number of tests which use windows L&F were marked as
windows-specific
<br>
- The stub "ThemeReader.java" which was used to build win
w&f on unix was removed.
<br>
<br>
[1] <a class="moz-txt-link-freetext" href="https://bugs.openjdk.java.net/browse/JDK-6461834">https://bugs.openjdk.java.net/browse/JDK-6461834</a>
<br>
[2] <a class="moz-txt-link-freetext" href="https://bugs.openjdk.java.net/browse/JDK-8184813">https://bugs.openjdk.java.net/browse/JDK-8184813</a>
<br>
[3] <a class="moz-txt-link-freetext" href="https://bugs.openjdk.java.net/browse/JDK-8075255">https://bugs.openjdk.java.net/browse/JDK-8075255</a>
<br>
<br>
</blockquote>
<br>
</blockquote>
<br>
</body>
</html>