[FFmpeg-devel] [PATCH] Add VorbisComment writing to ogg-vorbis files (bump)

Baptiste Coudurier baptiste.coudurier
Wed May 5 01:23:58 CEST 2010

On 04/18/2010 03:47 AM, James Darnley wrote:
> On 8 April 2010 15:39, James Darnley<james.darnley at gmail.com>  wrote:
>> On 31 March 2010 00:35, James Darnley<james.darnley at gmail.com>  wrote:
>>>> As the subject says, the attached patch adds VorbisComment writing to ogg-vorbis files.
>>> Perhaps more explaination will garner some attention.
>>> The change to ogg_write_vorbiscomment() is because vorbis requires
>>> VorbisComments to have the framing bit whereas flac and speex (based
>>> on inspection of a file made by speexenc) do not.
>>> Any other comments about the way I've done stuff?
>> New patch attached.  Changed due to the recent AVERROR() changes.  A
>> couple of minor cosmetic changes too.
> bump...  does nobody want this?

It would help review it the attachement had a better mime type.
There is an extra space after framing_bit and before the ')'.

How is vorbis different from theora ? Also it seems that all codecs can 
write a vorbiscomment so the code should be factorized.

Key fingerprint                 8D77134D20CC9220201FC5DB0AC9325C5C1ABAAA
FFmpeg maintainer                                  http://www.ffmpeg.org

More information about the ffmpeg-devel mailing list