[FFmpeg-devel] [PATCH] Ogg: discard non-essential metadata from Vorbis header when creating extradata.
Måns Rullgård
mans
Mon Jan 31 21:23:37 CET 2011
Reimar D?ffinger <Reimar.Doeffinger at gmx.de> writes:
> On Mon, Jan 31, 2011 at 06:30:02PM +0000, M?ns Rullg?rd wrote:
>> Reimar D?ffinger <Reimar.Doeffinger at gmx.de> writes:
>> > What would be the point of a realloc besides wasting CPU time?
>> > The point is that priv->len is smaller and thus the amount
>> > of data that is copied into extradata is smaller.
>> > I can't imagine we have to optimize for a few 100 kB during header
>> > parsing.
>>
>> You're right of course. I overlooked the second copying. Please send
>> a properly formatted patch and we'll apply it.
>
> I'd do that more often if I had an efficient way to make git work
> with a mail-client instead of it trying to be (a really sucky) one.
> The idiocy of having the commit message in the subject line will
> probably reliably prevent that though.
You may attach patches created with git format-patch if you prefer.
That's no more or less work than creating diffs with svn.
> --8<--
> The first part of the metadata, the "vendor" string, is required by
> libvorbis, it will refuse to play when it is not available.
> Also we do not currently parse that part into metadata so it would also
> be lost if we removed it as well.
> ---
> libavformat/oggparsevorbis.c | 12 ++++++++++--
> 1 files changed, 10 insertions(+), 2 deletions(-)
Thanks, pushed.
--
M?ns Rullg?rd
mans at mansr.com
More information about the ffmpeg-devel
mailing list