[FFmpeg-devel] [PATCH 1/3] lavfi: remove default config_props() callback and refactor avfilter_config_links()

[Stefano Sabatini]

On date Thursday 2011-08-04 19:39:48 +0200, Stefano Sabatini encoded:
> On date Thursday 2011-08-04 19:22:01 +0300, Mina Nagy Zaki encoded:
> > On Thu, Aug 04, 2011 at 07:05:22PM +0200, Stefano Sabatini wrote:
> > > On date Thursday 2011-08-04 15:47:00 +0300, Mina Nagy Zaki encoded:
> > [...]
> > > 
> > > Looks fine, but you should also remove the declaration in avfilter.h
> > > (BTW I noticed that avfilter_default_config_input_link() is never
> > > defined).
> > 
> > The config_props() for inputs is just used to allow filters to react to getting
> > linked, a default handler is meaningless.
> 
> > I have removed its declaration as well.
> 
> OK, though it deserves a separate patch (no need to send it, I'll do
> it when applying this one).
> 
> > From 2987659729f7ee7595c83e6b013e5d1a2db7ef2f Mon Sep 17 00:00:00 2001
> > From: Mina Nagy Zaki <mnzaki at gmail.com>
> > Date: Thu, 4 Aug 2011 12:34:05 +0300
> > Subject: [PATCH] lavfi: remove default config_props() callback and refactor
> >  avfilter_config_links()
> > 
> > Link properties have to be checked after config_props() is called to make sure
> > everything is sane, so the default config_props() for output links was
> > redundant, and for input links it is not required (and was never defined)
> > 
> > Also removed channel_layout check as it is now negotiated.
> > ---
> >  libavfilter/avfilter.c |   59 +++++++++++++++++++++++++++++++++++-------------
> >  libavfilter/avfilter.h |    6 -----
> >  libavfilter/defaults.c |   23 ------------------
> >  3 files changed, 43 insertions(+), 45 deletions(-)
> 
> Looks fine to me, going to apply in a few days if I see no comments.

Applied.
-- 
FFmpeg = Forgiving and F*****g Mean Problematic Eager God