Alex Menkov wrote:

Karl Helgason wrote:
WaveExtensibleFileReader,WaveFloatFileWriter,WaveFloatFileReader has been moved to src.floatwav in the Gervill CVS. Because they aren't directly related with Gervill Synthesizer. They simply add support for float encoded Wave files.
And WaveExtensibleFileReader is also a proposal how we should store
channel mask in AudioFormat properties (used for multi-channel wave files).
So, should I add these files to OpenJDK 6?


It depends, Alexey thought we should rather add support for PCM Float support
to current com.sun.media.sound.WaveFileReader and WaveFileWriter classes
than introducing new one.

All this wave readers (current PCM/ALAW/ULAW and Float & Extensible from Gervill) could be combined in different ways - basing on current WaveFileReader or basing on WaveFloatFileReader or WaveExtensibleFileReader.
Main goal is to avoid functionality duplication.

I'm personally agnostic toward how this functionality is implemented. Alex and Karl, how about I add these files along with the other changes I'm making?

-Joe


>>> And WaveExtensibleFileReader is also a proposal how we should store
>>> channel mask in AudioFormat properties (used for multi-channel wave files).

Wow.. I've overlooked this :)



Regards
Alex

Reply via email to