[FFmpeg-devel] [PATCH 3/3] ffmpeg: support audio in complex filters

Michael Niedermayer michaelni at gmx.at
Mon May 7 18:52:31 CEST 2012


On Mon, May 07, 2012 at 06:27:11PM +0200, Nicolas George wrote:
> Le nonidi 19 floréal, an CCXX, Matthieu Bouron a écrit :
> > The solution i provided in previous mail with missing patch introduce two new
> > functions in ffmpeg.c:
> >   graph_prepend_filter,
> >   graph_create_buffer_filter wich is basically the same as
> >   avfilter_graph_create_filter but use graph_preprend_filter.
> > 
> > IMO this has the advantage to be local to ffmpeg but duplicates codes from
> > avfilter. Maybe it's better to provide a patch to avfiltergraph and insert
> > filter with no input at begin of the array ?
> 
> I saw it, and you summarize the differences pretty well.
> 
> I am personally rather for the solution of modifying
> avfilter_graph_add_filter, because that makes less total code and separate
> patches, an possibly useful for other situations. But maybe I am biased
> because I suggested it. So better wait other advices.

iam happy with any solution that works and that others are happy with

[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Let us carefully observe those good qualities wherein our enemies excel us
and endeavor to excel them, by avoiding what is faulty, and imitating what
is excellent in them. -- Plutarch
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20120507/58a025cc/attachment.asc>


More information about the ffmpeg-devel mailing list