[FFmpeg-devel] [PATCH] ffmpeg.c: disable av_vsrc_buffer_add_frame() when buffer filter is disabled

[Peter Ross]

On Sun, May 30, 2010 at 04:42:51AM +0200, Michael Niedermayer wrote:
> On Sun, May 30, 2010 at 12:39:35PM +1000, Peter Ross wrote:
> > On Sun, May 30, 2010 at 02:46:40AM +0100, M?ns Rullg?rd wrote:
> > > Peter Ross <pross at xvid.org> writes:
> > > 
> > > > $subject
> > > >
> > > > -- Peter
> > > > (A907 E02F A6E5 0CD2 34CD 20D2 6760 79C5 AC40 DD6B)
> > > >
> > > > diff --git a/ffmpeg.c b/ffmpeg.c
> > > > index 63f0e6b..b0dfcff 100644
> > > > --- a/ffmpeg.c
> > > > +++ b/ffmpeg.c
> > > > @@ -1651,7 +1651,7 @@ static int output_packet(AVInputStream *ist, int ist_index,
> > > >                                      &buffer_to_free);
> > > >          }
> > > >  
> > > > -#if CONFIG_AVFILTER
> > > > +#if CONFIG_AVFILTER && CONFIG_BUFFER_FILTER
> > > >          if (ist->st->codec->codec_type == AVMEDIA_TYPE_VIDEO && ist->input_video_filter) {
> > > >              // add it to be filtered
> > > >              av_vsrc_buffer_add_frame(ist->input_video_filter, &picture,
> > > 
> > > This doesn't work.
> > 
> > Okay. So what are our options?
> > 
> > 1. require the buffer filter to always be present when libavfilter is enabled
> 
> sounds reasonable

Fix enclosed.

-- Peter
(A907 E02F A6E5 0CD2 34CD 20D2 6760 79C5 AC40 DD6B)
-------------- next part --------------
A non-text attachment was scrubbed...
Name: ffmpeg-en-buffer-filter.diff
Type: text/x-diff
Size: 327 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20100530/378f4d33/attachment.diff>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20100530/378f4d33/attachment.pgp>