<Sound Dev> [9] Review Request: 8064800 AudioSystem/WaveFileWriter can't write PCM_FLOAT, but writes it anyway

Sergey Bylokhov Sergey.Bylokhov at oracle.com
Thu Jan 7 20:49:12 UTC 2016


Hello, Audio Guru.

Please review the fix for jdk9.

Support of the PCM_FLOAT encoding was added to the jdk7, but some 
important things were forgotten:
  - The new WaveFloatFileWriter.java was not added to the 
./services/javax.sound.sampled.spi.AudioFileWriter, which means that it 
is never used by the java sound.
- The old writers(AiffFileWriter, AuFileWriter, WaveFileWriter) was 
written in an assumption that they support all possible encoding.

Notes:
  - In the fix I reused isFileTypeSupported() method which checks the 
encoding of the stream.
  - In the test some of the data are commented or skipped: for example 
we support float&32bits only but we do not check this in the 
isFileTypeSupported. Same for "channels = AudioSystem.NOT_SPECIFIED" the 
isFileTypeSupported() returns true but write() method fails, etc. I plan 
to fix it separately.

+cleanup of the related methods in AudioSystem.java.

Bug: https://bugs.openjdk.java.net/browse/JDK-8064800
Webrev can be found at: http://cr.openjdk.java.net/~serb/8064800/webrev.00

-- 
Best regards, Sergey.


More information about the sound-dev mailing list