[MPlayer-cvslog] r24539 - trunk/libmpdemux/demux_lavf.c

[Benjamin Zores]

Reimar Döffinger a écrit :
> Hello,
> On Sat, Sep 15, 2007 at 10:17:23PM +0200, ben wrote:
>> Author: ben
>> Date: Sat Sep 15 22:17:22 2007
>> New Revision: 24539
>>
>> Log:
>> added monkey audio fourcc in wave header
>>
>> Modified:
>>    trunk/libmpdemux/demux_lavf.c
>>
>> Modified: trunk/libmpdemux/demux_lavf.c
>> ==============================================================================
>> --- trunk/libmpdemux/demux_lavf.c	(original)
>> +++ trunk/libmpdemux/demux_lavf.c	Sat Sep 15 22:17:22 2007
>> @@ -78,6 +78,7 @@ static const AVCodecTag mp_wav_tags[] = 
>>      { CODEC_ID_ADPCM_IMA_WS,      MKTAG('A', 'I', 'W', 'S')},
>>      { CODEC_ID_ADPCM_THP,         MKTAG('T', 'H', 'P', 'A')},
>>      { CODEC_ID_AMR_NB,            MKTAG('n', 'b',   0,   0)},
>> +    { CODEC_ID_APE,               MKTAG('M', 'A', 'C', ' ')},
> 
> For what is this necessary? Not for the demuxer since lavf sets
> codec_tag (whether or not that's the right behaviour is another
> question). The bigger problem is that this value conflicts with both
> what lavf sets and what is in etc/codecs.conf, which is "APE ".

I'll remove then.
By the way, why is fourcc set to "APE " in codecs.conf while its 
official one should be "MAC " ??

Ben