-
Bug
-
Resolution: Fixed
-
P2
-
1.3.0
-
None
-
rc2
-
generic
-
generic
From: ###@###.###
To: jdk-kestrel-core@eng
cc: ccc@eng, kkytle@eng, mjb@eng, abennett@eng
Subject: Unapproved javax.sound API change in Kestrel (4303051, 4303617)
Date: Wed, 08 Mar 2000 09:31:40 -0800
The fix for these bugs resulted in an inadvertent API change due to a design
flaw in the javax.sound.sampled.AudioInputStream class. That class contains a
protected stream field, which really should be private. The type of the field
was changed from InputStream to PushbackInputStream as part of the fix, hence
the API change. Had the field been private then this would not have happened.
The best solution to this problem from a technical standpoint is to change the
stream field in the AudioInputStream class from protected to private. This
would both fix the current problem and prevent similar problems from occurring
in the future. I strongly suspect that this can be done with a few one-line
changes to AudioInputStream and its three subclasses in the com.sun.media.sound
package. I have made these changes in a private 1.3 workspace and have started
a build to verify this theory. I'll send a followup note once the build is
complete.
To: jdk-kestrel-core@eng
cc: ccc@eng, kkytle@eng, mjb@eng, abennett@eng
Subject: Unapproved javax.sound API change in Kestrel (4303051, 4303617)
Date: Wed, 08 Mar 2000 09:31:40 -0800
The fix for these bugs resulted in an inadvertent API change due to a design
flaw in the javax.sound.sampled.AudioInputStream class. That class contains a
protected stream field, which really should be private. The type of the field
was changed from InputStream to PushbackInputStream as part of the fix, hence
the API change. Had the field been private then this would not have happened.
The best solution to this problem from a technical standpoint is to change the
stream field in the AudioInputStream class from protected to private. This
would both fix the current problem and prevent similar problems from occurring
in the future. I strongly suspect that this can be done with a few one-line
changes to AudioInputStream and its three subclasses in the com.sun.media.sound
package. I have made these changes in a private 1.3 workspace and have started
a build to verify this theory. I'll send a followup note once the build is
complete.
- relates to
-
JDK-4303617 Animator Demos produce a coarse staticy sound when run on browsers w/ JRE RC2 W
-
- Closed
-
-
JDK-4303051 AudioInputStream.read doesn't retry to ensure even # of frames
-
- Closed
-