[FFmpeg-devel] [PATCH] New registration system for avfilter filters

Stefano Sabatini stefano.sabatini-lala
Wed Nov 25 00:50:26 CET 2009


On date Monday 2009-11-23 23:05:29 +0100, Michael Niedermayer encoded:
> On Mon, Nov 23, 2009 at 10:49:52PM +0100, Stefano Sabatini wrote:
> > On date Monday 2009-11-23 20:40:01 +0100, Michael Niedermayer encoded:
> > > On Mon, Nov 23, 2009 at 01:37:20AM +0100, Stefano Sabatini wrote:
> > > [...]
> > > > +AVFilter *registered_avfilters[MAX_REGISTERED_AVFILTERS_NB + 1];
> > > > +
> > > > +int next_registered_avfilter_idx = 0;
> > > 
> > > static 
> > 
> > Fixed.
> >  
> > > >  AVFilter *avfilter_get_by_name(const char *name)
> > > >  {
> > > > -    AVFilter *filter;
> > > > +    int i;
> > > >  
> > > > -    for (filter = first_avfilter; filter; filter = filter->next)
> > > > -        if (!strcmp(filter->name, name))
> > > > -            return filter;
> > > 
> > > > +    for (i = 0; registered_avfilters[i] && i < MAX_REGISTERED_AVFILTERS_NB; i++)
> > > 
> > > redundant check
> > 
> > Yes.
> > 
> > Patch updated.
> > -- 
> > FFmpeg = Friendly and Fantastic Mournful Pure Erudite God
> 
> >  avfilter.c |   29 ++++++++++++++++-------------
> >  avfilter.h |    4 +++-
> >  2 files changed, 19 insertions(+), 14 deletions(-)
> > 64518342d929736366f462a53c425cb9cb1120d1  change-avfilter-register.patch
> 
> ok

Applied.
-- 
FFmpeg = Foolish and Foolish Marvellous Puristic Eretic Gadget



More information about the ffmpeg-devel mailing list