[Ffmpeg-devel] [PATCH] AMR demuxer fourcc

Michael Niedermayer michaelni
Sun Apr 24 12:46:12 CEST 2005


Hi

On Saturday 23 April 2005 22:27, Roberto Togni wrote:
> Hi all
>
> amr.c demuxer sets the codec tag (fourcc) to CODEC_ID_AMR_[NB|WB],
> instead of using the common fourcc "samr" and "sawb" or leaving it
> blank.
>
> This make .amr files umplayable in mplayer, unless you add the codec ids
> as format 0x12000 and format 0x12001 in codecs.conf; but i don't like
> using internal codec id as format identifiers.
>
> Other demuxers that do not use fourcc leave the tag field = 0, and
> mplayer demux_lavf fill it with codec_get_wav_tag (or bmp for video),
> but it won't work in this case because there is no known twocc for amr
> audio.
>
> The attached patch make amr.c set the tag field to the official fourcc
> for amr nb and wb. Ok to apply?

ok, its certainly better then whats currently in cvs ...

[...]
-- 
Michael

"nothing is evil in the beginning. Even Sauron was not so." -- Elrond





More information about the ffmpeg-devel mailing list