On Mon, Jun 25, 2012 at 12:07:57PM +0200, Anton Khirnov wrote:
> On Mon, 25 Jun 2012 10:42:12 +0200, Diego Biurrun <[email protected]> wrote:
> > On Fri, Jun 22, 2012 at 09:35:45PM +0200, Anton Khirnov wrote:
> > > 
> > > --- a/libavfilter/avfilter.c
> > > +++ b/libavfilter/avfilter.c
> > > @@ -443,11 +443,7 @@ enum AVMediaType avfilter_pad_get_type(AVFilterPad 
> > > *pads, int pad_idx)
> > >  
> > > -#if FF_API_DEFAULT_CONFIG_OUTPUT_LINK
> > > -int avfilter_default_config_output_link(AVFilterLink *link)
> > > -{
> > > -    return 0;
> > > -}
> > > +#if FF_API_FILTERS_PUBLIC
> > >  void avfilter_insert_pad(unsigned idx, unsigned *count, size_t 
> > > padidx_off,
> > >                           AVFilterPad **pads, AVFilterLink ***links,
> > >                           AVFilterPad *newpad)
> > 
> > This looks like a separate change to me.
> 
> Technically it is, but it seemed silly to have a separate patch fixing
> that only to remove it completely in a following commit.

Please split it off into a separate commit as that will make for a more
sensible history, thank you.

Diego
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to