[MPlayer-dev-eng] [PATCH] lavf demuxer: always overwrite codec_tag for PCM formats

Roberto Togni rxt at rtogni.it
Sat Jan 19 15:21:14 CET 2008


On Sun, 13 Jan 2008 18:40:21 +0100
Roberto Togni <rxt at rtogni.it> wrote:

> Hi all,
>  currently demuxer_lavf fails to set the correct codec format for some
> pcm files (eg. .au files, see samples/au/vrooom.au). This happens
> because the demuxer fills codec_tag with the tag used in the file (2
> for vrooom.au), and so MPlayer does not look into the codec_id =>
> codec_tag table to fix it. Also in this case the original codec_tag is
> already used by MPlayer (ms dpcm).
> lavf demuxer set the codec_id to the right format for pcm codecs.
> 
> This patch forces demuxer_lavf to ignore the codec_tag for pcm formats,
> and to always replace it with a tag computed from the CODEC_ID_PCM_*
> value.
> 
> It fixes the vrooom.au file and works with the other files I've tested.
> 

Updated version attached. I'll apply it soon if no complaints.

Ciao,
 Roberto

-- 
Better is the enemy of good enough.



More information about the MPlayer-dev-eng mailing list