<html>
<head>
<meta content="text/html; charset=utf-8" http-equiv="Content-Type">
</head>
<body bgcolor="#FFFFFF" text="#000000">
<pre><span class="changed">Looks OK except for a couple of typo nits below. You can fix those without a new review
However you will need a CCC for deprecating the public API AppletInitializer
and the Beans.instantiate method which references it
159 * @deprecated It s recommended to use</span></pre>
"It is" ?<br>
<br>
--<br>
<pre><span class="changed">160 * {@link #instantiate(ClassLoader, String, BeanContext)}, because the</span>
<span class="changed"> 161 * Applet API is deprecated. See the</span></pre>
<br>
Nit picky, but since you need to edit this anyway please break <br>
the above in the source as<br>
<pre><span class="changed">160 * {@link #instantiate(ClassLoader, String, BeanContext)},</span>
<span class="changed">161 * </span><span class="changed"><span class="changed">because the</span> Applet API is deprecated. See the</span></pre>
<br>
-phil.<br>
<br>
<div class="moz-cite-prefix">On 11/17/2016 09:31 AM, Sergey Bylokhov
wrote:<br>
</div>
<blockquote
cite="mid:fc5b4b38-c374-9063-045f-2ccf0d6de929@oracle.com"
type="cite">Hello.
<br>
<br>
Please review the fix for jdk9.
<br>
<br>
Some time ago the Applets were deprecated,see JEP-289[1]. After
that the code which uses Applets were marked with
"@SuppressWarnings("deprecation")"[2].
<br>
<br>
This is a request to deprecate such API in JavaBeans and in Swing.
<br>
I also mark as deprecated the code which is related to
AppletViewer(it is also deprecated[3])
<br>
<br>
[1] <a class="moz-txt-link-freetext" href="http://openjdk.java.net/jeps/289">http://openjdk.java.net/jeps/289</a>
<br>
[2]
<a class="moz-txt-link-freetext" href="http://mail.openjdk.java.net/pipermail/swing-dev/2016-October/006788.html">http://mail.openjdk.java.net/pipermail/swing-dev/2016-October/006788.html</a><br>
[3] <a class="moz-txt-link-freetext" href="https://bugs.openjdk.java.net/browse/JDK-8074165">https://bugs.openjdk.java.net/browse/JDK-8074165</a>
<br>
<br>
Bug: <a class="moz-txt-link-freetext" href="https://bugs.openjdk.java.net/browse/JDK-8169900">https://bugs.openjdk.java.net/browse/JDK-8169900</a>
<br>
Webrev can be found at:
<a class="moz-txt-link-freetext" href="http://cr.openjdk.java.net/~serb/8169900/webrev.00">http://cr.openjdk.java.net/~serb/8169900/webrev.00</a>
<br>
<br>
</blockquote>
<br>
</body>
</html>