<html>
  <head>
    <meta content="text/html; charset=utf-8" http-equiv="Content-Type">
  </head>
  <body bgcolor="#FFFFFF" text="#000000">
    <p>Do we need the suppression in JavaSoundAudioClip.java as I could
      not find any mention of Applet in that class?<br>
    </p>
    <span class="changed">In AppletViewer.java</span> 
    <br>
    shouldn't we need to add "deprecation" to <br>
    <pre><span class="changed">45 @SuppressWarnings("serial") // JDK implementation class 


</span>Do we still need this

160     @SuppressWarnings("deprecation")
since we added 

<span class="changed">120 @SuppressWarnings({"serial", "deprecation"})</span>

Other than that, looks good to me.

Regards
Prasanta
</pre>
    <div class="moz-cite-prefix">On 10/20/2016 1:44 AM, Philip Race
      wrote:<br>
    </div>
    <blockquote cite="mid:5807D43A.10409@oracle.com" type="cite">Bug:
      <a class="moz-txt-link-freetext" href="https://bugs.openjdk.java.net/browse/JDK-8168316">https://bugs.openjdk.java.net/browse/JDK-8168316</a>
      <br>
      Webrev: <a class="moz-txt-link-freetext" href="http://cr.openjdk.java.net/~prr/8168316/">http://cr.openjdk.java.net/~prr/8168316/</a>
      <br>
      <br>
      When applets were deprecated it seems that due to all deprecated
      <br>
      warning being suppressed in java.desktop many places that should
      <br>
      have been updated weren't
      <br>
      <br>
      This resolves the applet ones .. as a precursor to fixing up the
      other issues
      <br>
      that cause this global supression.
      <br>
      <br>
      -phil
      <br>
    </blockquote>
    <br>
  </body>
</html>