On Sun, Nov 30, 2014 at 12:11:07PM +0100, Christophe Gisquet wrote: > 2014-11-30 10:59 GMT+01:00 Christophe Gisquet <christophe.gisq...@gmail.com>: > > So I'm going to add a "AVDictionary *bsf_args" to OutputStream. > > > > That dictionary lookup seems negligible to the amount of work done overall. > > Done in the attached patch. I've edited the documentation also to > reflect this change that only affects ffmpeg. > > -- > Christophe
> doc/bitstream_filters.texi | 8 ++++++++ > ffmpeg.c | 7 ++++++- > ffmpeg.h | 1 + > ffmpeg_opt.c | 4 ++++ > 4 files changed, 19 insertions(+), 1 deletion(-) > b5afd68b547a0b9df1963a1f528d6c03541508a8 > 0001-ffmpeg-take-bsf-arguments-from-the-command-line.patch > From cccecb1eef2e387115d83ec5bad58e006f4007ea Mon Sep 17 00:00:00 2001 > From: Christophe Gisquet <christophe.gisq...@gmail.com> > Date: Sat, 29 Nov 2014 19:15:02 +0100 > Subject: [PATCH 1/4] ffmpeg: take bsf arguments from the command line > > The format is now: > -bsf:X filter1[=opt1=str1/opt2=str2],filter2 > ie the parameters are appended after the filter name using '='. As ',' > has been reserved already for the list of filters, '/' is just an > example of token separation for now, but that could become part of the > API to avoid each bsf using its own tokenization. > > The proper solution would be using AVOption, but this is overkill for now. > --- > doc/bitstream_filters.texi | 8 ++++++++ > ffmpeg.c | 7 ++++++- > ffmpeg.h | 1 + > ffmpeg_opt.c | 4 ++++ > 4 files changed, 19 insertions(+), 1 deletion(-) applied thanks [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB Democracy is the form of government in which you can choose your dictator
signature.asc
Description: Digital signature
_______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel