[FFmpeg-devel] [PATCH 1/7] lavfi: add a closed field to AVFilerLink.

[Michael Niedermayer]

On Tue, Aug 21, 2012 at 08:21:53PM +0200, Stefano Sabatini wrote:
> On date Tuesday 2012-08-21 09:42:25 +0200, Nicolas George encoded:
> > Le quintidi 5 fructidor, an CCXX, Stefano Sabatini a écrit :
[...]
> > > So I suppose this can be used also for *opening* a link?
> > 
> > Provided it has been tested and work, it may become useful at tome point. It
> > does not cost anything to have the API ready if the needs should arise. In
> > fact, if it were not for ABI compatibility with the fork, we would let
> > applications set the field directly.
> > 
> > > Looks good to me, but I'm not sure which specific problem it is
> > > addressing.
> > 
> > When an application is no longer interested by the output of a filter (like
> > ffmpeg with the -t option), but still interested by the output of another,
> > it is nice to be able to stop the expensive processing for frames that will
> > be immediately discarded anyway.
> 
> Thanks for the explanation. I'm fine with the patch, but Michael may
> want to have a look at it and comment.

Iam fine with adding a closed flag as well.
It will require more than local per filter closing propagation though
in case of loops.
that is set all links to closed and then from the outputs that are
still open go over the graph backward and reopen all links, should
work

[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

The real ebay dictionary, page 1
"Used only once"    - "Some unspecified defect prevented a second use"
"In good condition" - "Can be repaird by experienced expert"
"As is" - "You wouldnt want it even if you were payed for it, if you knew ..."
-------------- 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/20120822/23e901f6/attachment.asc>