[Ffmpeg-cvslog] r6963 - trunk/libavformat/riff.c

[Måns Rullgård]

Oded Shimon <ods15 at ods15.dyndns.org> writes:

> On Sat, Nov 11, 2006 at 03:08:40AM +0100, Michael Niedermayer wrote:
>> Hi
>> 
>> On Sat, Nov 11, 2006 at 01:22:08AM +0000, M?ns Rullg?rd wrote:
>> > michael <subversion at mplayerhq.hu> writes:
>> > 
>> > > Author: michael
>> > > Date: Sat Nov 11 01:29:49 2006
>> > > New Revision: 6963
>> > >
>> > > Modified:
>> > >    trunk/libavformat/riff.c
>> > >
>> > > Log:
>> > > libnut uses "MP3 " for mp3
>> > >
>> > > Modified: trunk/libavformat/riff.c
>> > > ==============================================================================
>> > > --- trunk/libavformat/riff.c	(original)
>> > > +++ trunk/libavformat/riff.c	Sat Nov 11 01:29:49 2006
>> > > @@ -200,6 +200,7 @@
>> > >      // for NuppelVideo (nuv.c)
>> > >      { CODEC_ID_PCM_S16LE, MKTAG('R', 'A', 'W', 'A') },
>> > >      { CODEC_ID_MP3, MKTAG('L', 'A', 'M', 'E') },
>> > > +    { CODEC_ID_MP3, MKTAG('M', 'P', '3', ' ') },
>> > 
>> > Nooooo.  We were trying to get rid of those invalid bogus tags from
>> > riff.c.  Don't add more of them.
>> 
>> nut per spec uses the avi-riff tags to identify codecs, libnut uses
>> "MP3 " ...
>> ask oded why
>
> I actually don't know myself. Ask Rich, he told me to do it. :/ BTW, I use 
> "mp3 " (no caps).

See?  An official list would avoid this kind of confusion.

-- 
M?ns Rullg?rd
mru at inprovide.com