AW: [Clam-devel] [PATCH] DEBUG-related, PortAudio, SinTracking, AudioFileFormats, SConstruct

David García Garzón dgarcia at iua.upf.edu
Thu Jun 14 09:22:56 PDT 2007


On Thursday 14 June 2007 10:15:35 Rettenbacher, Bernhard wrote:
> Hi Pau,
>
> AudioFileFormats patch:
> I really do not know the exact cause for changing this (it was 2 months
> ago, I did it ). One cause I remember is the different number of entries in
> the extensionMap struct (19) and the number of entries in EAudioFileFormat
> (16). There was an error when accessing extensionMap elements > 15.

Hi, Bernhard
Thanks for the patches

Here we were using a trick we use all around clam of doing arrays of 
structures with the last element having a null. We use that trick to avoid 
the typical and hard to check error of setting the size wrongly and having to 
update it every time you add or remove an element.

So as we use it all around CLAM we want to know why it failed, so maybe we 
should take a different approach. Or, in the case it wasn't the error, 
avoiding to go back to the numbered style.

Could you revert the change and find again why you had to change this? It is 
possible that the real error could be in a different place.

Thanks.
David.





More information about the clam-devel mailing list