[FFmpeg-soc] [PATCH] Move format from AVFilterBuffer to AVFilterPicRef

S.N. Hemanth Meenakshisundaram smeenaks at ucsd.edu
Fri Aug 6 19:21:42 CEST 2010


> > On Sun, Aug 01, 2010 at 11:41:09PM -0700, S.N. Hemanth
Meenakshisundaram wrote:
> > >  ffplay.c                  |    8 ++++----
> > >  libavfilter/avfilter.h    |    2 +-
> > >  libavfilter/defaults.c    |    8 ++++----
> > >  libavfilter/vf_overlay.c  |   12 ++++++------
> > >  libavfilter/vsrc_buffer.c |    2 +-
> > >  libavfilter/vsrc_movie.c  |    2 +-
> > >  6 files changed, 17 insertions(+), 17 deletions(-)
> > > 415376236550b0dd86fe180594d1f65f9d9d077f 
0001-Move-format-from-AVFilterBuffer-to-AVFilterPicRef.patch
> >
> > looks ok if tested

> This is causing regressions here.

> Hemanth, can you check it?

Hi Stefano,

The regression was being caused by my previous patch to resize
AVFilterBuffer's data and linesize arrays to 8. Strangely, the errors only
show up in make test after applying the current patch.

Applying the attached fix and then the current patch will solve the issue
and make test passes. I had only tested with make lavfitest and so missed
this. Will test with 'make test' too in the future.

Also, this bug fix will no longer be needed once the later patch to resize
AVFilterBufferRef itself is applied.

Regards,
Hemanth
-------------- next part --------------
A non-text attachment was scrubbed...
Name: fix.diff
Type: text/x-patch
Size: 500 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-soc/attachments/20100806/5bd5a2b2/attachment.bin>


More information about the FFmpeg-soc mailing list