[FFmpeg-cvslog] r21850 - in trunk/libavformat: avi.c avienc.c metadata.c metadata.h mp3.c utils.c

[Mike Melanson]

M?ns Rullg?rd wrote:
> Justin Ruggles <justin.ruggles at gmail.com> writes:
> 
>> James Darnley wrote:
>>
>>> On 16 February 2010 17:32, rbultje <subversion at mplayerhq.hu> wrote:
>>>> Log:
>>>> Set lavf identification string globally in av_write_header(), rather
>>>> than inside the muxers. Remove special handling of "encoder" tags from
>>>> AVI and MP3 muxers.
>>> What are the chances of getting this reverted?  I ask because my "flac
>>> tags" patch already writes the LIBAVFORMAT_IDENT string in the
>>> VorbisComment as the vendor_string.  This means that the encoder tag
>>> is redundant.  I would also argue that it is wrong because libavcodec
>>> is the encoder.
>> From Xiph.org VorbisComment recommendations:
>> http://wiki.xiph.org/index.php/VorbisComment#ENCODER
>>
>> - The vendor string is usually used to store the name and version of the
>> underlying codec library
>> - The idea of ENCODER is to store the name of the user-visible
>> application, for example ffmpeg2theora.
> 
> I wouldn't trust an organisation that can't distinguish between
> containers and codecs on this matter.
> 
>> They also recommend using a URL in the ENCODER tag.
> 
> Which their own software doesn't do.

We're all just a bunch of Ogg-hating wankers, says Monty of Xiph:

http://blog.gingertech.net/2010/02/20/googles-challenges-of-freeing-vp8/#comments

:)

-- 
     -Mike Melanson