Re: [Gnash-commit] gnash ChangeLog libbase/FLVParser.cpp

[Sandro Santilli]

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!