[FFmpeg-devel] [PATCH] Implement frei0r source.

[Stefano Sabatini]

On date Tuesday 2010-11-30 01:04:18 +0100, Michael Niedermayer encoded:
> On Fri, Nov 26, 2010 at 12:44:45AM +0100, Stefano Sabatini wrote:
> > On date Thursday 2010-11-25 18:18:16 +0100, Michael Niedermayer encoded:
> > > On Wed, Nov 24, 2010 at 01:38:10AM +0100, Stefano Sabatini wrote:
[...]
> >  doc/filters.texi         |   29 +++++++++++
> >  libavfilter/Makefile     |    1 
> >  libavfilter/allfilters.c |    1 
> >  libavfilter/vf_frei0r.c  |  121 +++++++++++++++++++++++++++++++++++++++++++----
> >  4 files changed, 142 insertions(+), 10 deletions(-)
> > 9c9ff63c028ab37b7f7e1bbb30697fdcce5a0850  0001-Implement-frei0r-source.patch
> > From d9468ca65013593a6f97039bc5abca6bb6c7aae7 Mon Sep 17 00:00:00 2001
> > From: Stefano Sabatini <stefano.sabatini-lala at poste.it>
> > Date: Mon, 8 Nov 2010 00:43:44 +0100
> > Subject: [PATCH] Implement frei0r source.
> 
> [...]
> > +static int source_request_frame(AVFilterLink *outlink)
> > +{
> > +    Frei0rContext *frei0r = outlink->src->priv;
> > +    AVFilterBufferRef *picref = avfilter_get_video_buffer(outlink, AV_PERM_WRITE, outlink->w, outlink->h);
> > +    picref->video->pixel_aspect = (AVRational) {1, 1};
> > +    picref->pts = frei0r->pts++;
> 
> > +    picref->pos = -1;
> 
> shouldnt this be default?

No, currently the bufref returned by
avfilter_default_get_video_buffer() is zero-inited.

> otherwise it should be ok if tested

Applied (and possibly fixed fate).
-- 
FFmpeg = Fundamental & Friendly Meaningful Portable Enhanced Ghost