[MPlayer-dev-eng] [PATCH] amr support

[Roberto Togni]

On 2004.12.28 15:50, Richard van der Hoff wrote:
> Roberto Togni wrote:
>> On 2004.12.26 12:54, Reimar Döffinger wrote:
>> Try2, now nonsense options are accepted and no checks if --enable
>> Float version is preferred because it's default in ffmpeg
>>
> Roberto, I'm afraid this doesn't work for me. The problem is that,  
> not only is the nchannels field different, but also the sample-size  
> field - you do need those "samplebits" changes from my patch. The mov  
> demuxer actually assumes rather more about the header than is  
> specified - many of the fields are dependent on the actual codec in  
> use; it's just fortunate that they are conventially used in a certain  
> way - and this is one of the things I was trying to address by using  
> the 'switch' statement in my patch.
> 
> Do you not get satisfactory results by using the configure and codecs  
> part of your patch, together with the demux_mov part of my patch?
>

It worked with my samples without it, so i only applied the part with  
channels; now i'll apply the complete patch in the final version  
(Richard patch is available here  
http://bugzilla.mplayerhq.hu/attachment.cgi?id=71 ).

Can you upload a file that requires it so i can put it in the samples  
directory? And also a sample with amr_wb if you have it, i only have nb  
files.

Ciao,
 Roberto