[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Gnash-commit] gnash ChangeLog libbase/FLVParser.cpp
From: |
Sandro Santilli |
Subject: |
Re: [Gnash-commit] gnash ChangeLog libbase/FLVParser.cpp |
Date: |
Fri, 9 May 2008 19:24:25 +0200 |
On Fri, May 09, 2008 at 07:12:28PM +0200, Bastiaan Jacques wrote:
>
>
> On Fri, 9 May 2008, Sandro Santilli wrote:
> >>Some of ffmpeg's decoders use stream readers that will go past the end
> >>of the buffer you pass it (by design) because they read 64 bytes at a
> >>time. So we increase the buffer size to make sure they don't crash.
> >>(This is recommended by the ffmpeg docs.)
> >
> >Then rather then adding a fixed number (was 8, now is 64) we should
> >really add as much as to make the size a multiple of 64, right ?
>
> Sounds reasonable to me. Previously we just did exactly what ffmpeg docs
> suggested.
Any volunteer for a method like:
smallestMultipleOfGreaterOrEqualTo(base, atleast)
?
--strk;
() ASCII Ribbon Campaign
/\ Keep it simple!