[FFmpeg-devel] [PATCH] lavf/tee: add support for bitstream filtering

[Stefano Sabatini]

On date Thursday 2013-08-08 12:21:51 +0200, Stefano Sabatini encoded:
> On date Wednesday 2013-08-07 19:39:16 +0200, Nicolas George encoded:
> > L'octidi 18 thermidor, an CCXXI, Stefano Sabatini a écrit :
> > > Elaborating more on this. I'd need to store the couples
> > > specifiers+filters in a dictionary (this destroys the specification
> > > order and is a bit more complicate to implement). I leave the design
> > > choice to the maintainer.
> > 
> > Sorry for the delay, I was assuming you wanted to rework the patch first.
> > 
> > To answer that point, I do not see why you would need to store anything in a
> > dictionary. I believe it can be done with something like that:
> > 
> >     while (entry = av_opt_get(options, "bsfs", IGNORE_SUFFIX)) {
> > 	spec = entry->name + strlen("bsfs");
> > 	for (i = 0; i < nb_streams; i++) {
> > 	    if (stream matches stream specifier) {
> > 		if (stream[i]->bsfs) {
> > 		    av_log(WARNING, "Duplicate bsfs specification\n");
> > 		    continue;
> > 		}
> > 		parse_bsfs(stream[i], entry->value);
> > 	    }
> > 	    entry->value = NULL;
> > 	    av_opt_set(entry->name, NULL);
> > 	}
> >     }
> > 
> > Now that I think on it, I see the added benefit of matching the syntax of
> > ffmpeg itself for specifying bitstream filters:
> > 
> > ffmpeg ... -bsfs:a mp3_header_compress,noise -bsfs:v noise,mjpeg2jpeg ...
> 
> Check updated patch.
> -- 
> FFmpeg = Free Fierce Magical Political Ecstatic Gadget

> From 86c47df8afe3ee6b264272d25ec02da9814c3684 Mon Sep 17 00:00:00 2001
> From: Stefano Sabatini <stefasab at gmail.com>
> Date: Fri, 5 Jul 2013 18:33:30 +0200
> Subject: [PATCH] lavf/tee: add support for bitstream filtering
> 
> This allows to apply different bitstream filters to different outputs,
> with no need to trancode.
> 
> TODO: bump minor
> ---
>  doc/muxers.texi   |  19 ++++-
>  libavformat/tee.c | 217 +++++++++++++++++++++++++++++++++++++++++++++++++-----
>  2 files changed, 216 insertions(+), 20 deletions(-)

Ping.
-- 
FFmpeg = Frenzy Friendly Mythic Power Erudite Gadget