[FFmpeg-devel] [PATCH 1/6] lavf: add API to apply a list of bsfs to a packet

Michael Niedermayer michaelni at gmx.at
Thu Nov 19 15:05:19 CET 2015


On Wed, Nov 18, 2015 at 05:28:17PM -0600, Rodger Combs wrote:
> ---
>  libavformat/avformat.h |  8 ++++++++
>  libavformat/utils.c    | 49 +++++++++++++++++++++++++++++++++++++++++++++++++
>  2 files changed, 57 insertions(+)
> 
> diff --git a/libavformat/avformat.h b/libavformat/avformat.h
> index 5283085..96d3cfd 100644
> --- a/libavformat/avformat.h
> +++ b/libavformat/avformat.h
> @@ -2738,6 +2738,14 @@ int avformat_match_stream_specifier(AVFormatContext *s, AVStream *st,
>  
>  int avformat_queue_attached_pictures(AVFormatContext *s);
>  
> +/**
> + * Apply a list of bitstream filters to a packet.
> + *
> + * @return  >=0 on success;
> + *          AVERROR code on failure
> + */
> +int av_apply_bitstream_filters(AVFormatContext *s, AVPacket *pkt,
> +                               AVBitStreamFilterContext *bsfc);

is "s" the input or output context ?
is bsfc a NULL terminated list?
these should be documented

also at some point in the patch series version.h and APIChanges need
to be updated

should be ok otherwise

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Why not whip the teacher when the pupil misbehaves? -- Diogenes of Sinope
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 181 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20151119/c72c1a3f/attachment.sig>


More information about the ffmpeg-devel mailing list