[FFmpeg-devel] [PATCH] Add writing of vorbis comments to flac files

Justin Ruggles justin.ruggles
Sun Feb 14 21:36:54 CET 2010


James Darnley wrote:

> On 14 February 2010 02:13, Justin Ruggles <justin.ruggles at gmail.com> wrote:
>> What specifically did you try to do that did not work?  You should not
>> need to have year and author converted since the generic tags should
>> already be date and artist.
> 
> I have attached another patch which removes these two.  I was confused
> but a quick discussion on #ffmpeg-devel straightened me out.  (It
> would already be here if my ISP hadn't cut off my connection last
> night)
> 
>>> [...]
>>> +    /* The command line flac encoder defaults to placing a seekpoint
>>> +     * every 10s.  So one might add padding to allow that later
>>> +     * but there seems to be no simple way to get the duration here.
>>> +     * So let's try the flac default of 8192 bytes */
>>> +    flac_write_block_padding(s->pb, 8192, 1);
>>
>> I would recommend adding the writing of a padding block in a separate
>> commit.  It is unrelated to writing of VorbisComment tags, and it is the
>> cause of the change to the regression tests.
> 
> I could submit this as another patch but adding VorbisComment writing
> already changes the test results.

How does it change the test results other than that the last block flag
is no longer set because the padding comes after?  The regression tests
don't use any tags do they?

-Justin



More information about the ffmpeg-devel mailing list