<Sound Dev> JDK 9 RFR of JDK-8042871: Fix raw and unchecked warnings in sun.audio

Joe Darcy joe.darcy at oracle.com
Fri May 16 16:46:58 UTC 2014


On 05/15/2014 12:16 PM, Sergey Bylokhov wrote:
> Hi. Joe.
> It seems to me "Enumeration e" in AudioStreamSequence can be removed too.

Hi Sergey,

Actually it seems both variables in the class can be removed:

--- a/src/share/classes/sun/audio/AudioStreamSequence.java    Wed May 14 
18:19:14 2014 +0400
+++ b/src/share/classes/sun/audio/AudioStreamSequence.java    Fri May 16 
09:44:17 2014 -0700
@@ -44,16 +44,11 @@
   * @author Arthur van Hoff
   */
  public final class AudioStreamSequence extends SequenceInputStream {
-
-        Enumeration e;
-        InputStream in;
-
          /**
           * Create an AudioStreamSequence given an
           * enumeration of streams.
           */
-        public AudioStreamSequence(Enumeration e) {
+        public AudioStreamSequence(Enumeration<? extends InputStream> e) {
              super(e);
          }
-
-    }
+}

Updated webrev

     http://cr.openjdk.java.net/~darcy/8042871.1/

Thanks,

-Joe

>
> On 5/15/14 10:08 AM, Joe Darcy wrote:
>> Hello,
>>
>> Please review this fix for
>>
>>     JDK-8042871: Fix raw and unchecked warnings in sun.audio
>> http://cr.openjdk.java.net/~darcy/8042871.0/
>>
>> Patch below.
>>
>> Thanks,
>>
>> -Joe
>>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.openjdk.java.net/pipermail/sound-dev/attachments/20140516/cf488ccf/attachment-0001.html>


More information about the sound-dev mailing list