[FFmpeg-devel] [PATCH 1/6] bitstream_filter: add an init function

Michael Niedermayer michaelni at gmx.at
Sun Nov 30 13:46:43 CET 2014


On Sun, Nov 30, 2014 at 12:11:07PM +0100, Christophe Gisquet wrote:
> 2014-11-30 10:59 GMT+01:00 Christophe Gisquet <christophe.gisquet at 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.gisquet at 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
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 181 bytes
Desc: Digital signature
URL: <https://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20141130/ea043e36/attachment.asc>


More information about the ffmpeg-devel mailing list