[FFmpeg-devel] [PATCH 22/28] added: metadata support to oggenc with vorbis streams (submitted upstream Issue #555)

[James Darnley]

On 30 June 2010 11:09, Mans Rullgard <mans at mansr.com> wrote:
> From: Cory Fields <theuni-nospam- at xbmc.org>
>
> + ? ?if (m)
> + ? ? ? ?for(c = 0; c < m->count; ++c)
> + ? ? ? ? ? ?size += sizeof(uint32_t) + strlen(m->elems[c].key) + 1 + strlen(m->elems[c].value);
> + ? ?size += 1; // framing bit

That's what I originally did and people said that was wrong.

There is also my patch you may want to consider.