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

Michael Niedermayer michaelni
Mon Nov 23 23:05:29 CET 2009


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

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

The real ebay dictionary, page 2
"100% positive feedback" - "All either got their money back or didnt complain"
"Best seller ever, very honest" - "Seller refunded buyer after failed scam"
-------------- 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/20091123/da4168b4/attachment.pgp>



More information about the ffmpeg-devel mailing list