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
"It is" ?
--
160 * {@link #instantiate(ClassLoader, String, BeanContext)}, because the
161 * Applet API is deprecated. See the
Nit picky, but since you need to edit this anyway please break
the above in the source as
160 * {@link #instantiate(ClassLoader, String, BeanContext)},
161 * because the Applet API is deprecated. See the
-phil.
On 11/17/2016 09:31 AM, Sergey Bylokhov wrote:
Hello.
Please review the fix for jdk9.
Some time ago the Applets were deprecated,see JEP-289[1]. After that
the code which uses Applets were marked with
"@SuppressWarnings("deprecation")"[2].
This is a request to deprecate such API in JavaBeans and in Swing.
I also mark as deprecated the code which is related to AppletViewer(it
is also deprecated[3])
[1] http://openjdk.java.net/jeps/289
[2]
http://mail.openjdk.java.net/pipermail/swing-dev/2016-October/006788.html
[3] https://bugs.openjdk.java.net/browse/JDK-8074165
Bug: https://bugs.openjdk.java.net/browse/JDK-8169900
Webrev can be found at:
http://cr.openjdk.java.net/~serb/8169900/webrev.00
