[FFmpeg-devel] [PATCH 2/4] Use new av_dict_set_int helper function.

Michael Niedermayer michaelni at gmx.at
Tue Aug 12 12:05:12 CEST 2014


On Mon, Aug 11, 2014 at 09:17:17PM +0200, Reimar Döffinger wrote:
> Get rid of the many, slightly differing, implementations
> of basically the same thing.
> 
> Signed-off-by: Reimar Döffinger <Reimar.Doeffinger at gmx.de>
> ---
>  ffmpeg_opt.c                     | 12 +++---------
>  ffplay.c                         |  2 +-
>  libavfilter/vf_bbox.c            |  4 +---
>  libavfilter/vf_cropdetect.c      |  4 +---
>  libavformat/cinedec.c            |  4 +---
>  libavformat/ftp.c                | 10 ++++------
>  libavformat/hls.c                | 17 ++++-------------
>  libavformat/id3v1.c              |  4 +---
>  libavformat/matroskadec.c        | 31 +++++++------------------------
>  libavformat/mlvdec.c             | 16 ++++------------
>  libavformat/mov.c                | 14 +++-----------
>  libavformat/mxfdec.c             |  8 ++------
>  libavformat/smoothstreamingenc.c |  4 +---
>  libavformat/vqf.c                |  6 +-----
>  14 files changed, 34 insertions(+), 102 deletions(-)

nice simplification

LGTM from a quick look

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

Avoid a single point of failure, be that a person or equipment.
-------------- 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/20140812/519fb27c/attachment.asc>


More information about the ffmpeg-devel mailing list