[FFmpeg-devel] [PATCH] Add writing of vorbis comments to flac files
Sun Feb 14 21:41:18 CET 2010
Justin Ruggles wrote:
> 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
>>>> + /* 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?
Geez... nevermind. For some reason I thought that we already wrote an
empty comment tag, but we don't.
More information about the ffmpeg-devel