[Ffmpeg-devel] [PATCH] AMR demuxer fourcc

[Roberto Togni]

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?

Ciao,
 Roberto
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: amr.c.diff
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20050423/0309c8fd/attachment.asc>