Do we need the suppression in JavaSoundAudioClip.java as I could not find any mention of Applet in that class?

In AppletViewer.java
shouldn't we need to add "deprecation" to

45 @SuppressWarnings("serial") // JDK implementation class Do we still need this

160     @SuppressWarnings("deprecation")
since we added

120 @SuppressWarnings({"serial", "deprecation"})

Other than that, looks good to me.

Regards
Prasanta

On 10/20/2016 1:44 AM, Philip Race wrote:
Bug: https://bugs.openjdk.java.net/browse/JDK-8168316
Webrev: http://cr.openjdk.java.net/~prr/8168316/

When applets were deprecated it seems that due to all deprecated
warning being suppressed in java.desktop many places that should
have been updated weren't

This resolves the applet ones .. as a precursor to fixing up the other issues
that cause this global supression.

-phil

Reply via email to