[FFmpeg-devel] [PATCH] Increase ffmpeg video encoding buffer size to permit encoding of DPX images

Michael Niedermayer michaelni at gmx.at
Mon Mar 28 21:02:12 CEST 2011


On Mon, Mar 28, 2011 at 11:08:25PM +1100, Peter Ross wrote:
> The header generated by the DPX encoder is 0x680 bytes.
> ---
> On Sat, Mar 26, 2011 at 08:54:11AM +0100, Reimar Döffinger wrote:
> >
> >
> > On 26 Mar 2011, at 08:02, Peter Ross <pross at xvid.org> wrote:
> >
> > > The header generated by the DPX encoder is 0x680 bytes.
> > > ---
> > > ffmpeg.c |    2 +-
> > > 1 files changed, 1 insertions(+), 1 deletions(-)
> > >
> > > diff --git a/ffmpeg.c b/ffmpeg.c
> > > index 6f74e61..f0f6f50 100644
> > > --- a/ffmpeg.c
> > > +++ b/ffmpeg.c
> > > @@ -2243,7 +2243,7 @@ static int transcode(AVFormatContext **output_files,
> > >         }
> > >         if(codec->codec_type == AVMEDIA_TYPE_VIDEO){
> > >             int size= codec->width * codec->height;
> > > -            bit_buffer_size= FFMAX(bit_buffer_size, 6*size + 200);
> > > +            bit_buffer_size= FFMAX(bit_buffer_size, 6*size + 0x680);
> >
> > Should probably add a comment saying where the numbers come from.
> 
> Done.
> 
>  ffmpeg.c |    3 ++-
>  1 files changed, 2 insertions(+), 1 deletions(-)

LGTM

[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Frequently ignored awnser#1 FFmpeg bugs should be sent to our bugtracker. User
questions about the command line tools should be sent to the ffmpeg-user ML.
And questions about how to use libav* should be sent to the libav-user ML.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20110328/df2425ac/attachment.asc>


More information about the ffmpeg-devel mailing list