<Sound Dev> Updating Gervill for OpenJDK 6 build 12

[Joseph D. Darcy]

Karl Helgason wrote:
> Adding PCM Float support into current WaveFileReader.java, WaveFileWriter.java
> would be best. And it is just question of adding few lines.
> But until then I think you can use WaveFloatFileReader.java,  WaveFloatFileWriter.java.

Just to be clear, you recommend for now I add WaveFloatFileReader.java 
and  WaveFloatFileWriter.java. (from src.floatwav/ rather than src/) but 
  not include WaveExtensibleFileReader.java.

-Joe

> ________________________________________
> Frá: Joe.Darcy at Sun.COM [Joe.Darcy at Sun.COM]
> Sent: 29. júlí 2008 00:25
> Viðtakandi: Alex Menkov; Karl Helgason
> Afrit: sound-dev at openjdk.java.net; Alexey.Menkov at Sun.COM; Dalibor.Topic at Sun.COM
> Efni: Re: <Sound Dev> Updating Gervill for OpenJDK 6 build 12
> 
> 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
> 
> 
> 
> No virus found in this incoming message.
> Checked by AVG.
> Version: 7.5.524 / Virus Database: 270.5.6/1576 - Release Date: 27.7.2008 16:16
>