[FFmpeg-devel] [PATCH] optionally write QuickTime gama atom (was Re: Gamma function)

[Michael Niedermayer]

On Tue, Mar 03, 2015 at 11:10:54AM +0000, Kevin Wheatley wrote:
> Updated to use lrint()
> 
> On Mon, Mar 2, 2015 at 12:54 PM, Kevin Wheatley
> <kevin.j.wheatley at gmail.com> wrote:
> > Something like this - note this adds no tests, but fate still passes for me.
> >
> > Kevin

>  libavcodec/pngenc.c     |   29 +++-----------------------
>  libavformat/movenc.c    |   35 ++++++++++++++++++++++++++++++++
>  libavformat/movenc.h    |    2 +
>  libavutil/Makefile      |    2 +
>  libavutil/color_utils.c |   52 ++++++++++++++++++++++++++++++++++++++++++++++++
>  libavutil/color_utils.h |   29 ++++++++++++++++++++++++++
>  6 files changed, 124 insertions(+), 25 deletions(-)
> e5b12ceea12ce494d98c1bc30162a5e604c069c2  0001-Add-gama-atom-for-.mov-only-reuses-pngenc.c-gamma-va.patch
> From 794c8c3cf1e8506393fbcb4ddf1360042b0fc82f Mon Sep 17 00:00:00 2001
> From: Kevin Wheatley <kevin.j.wheatley at gmail.com>
> Date: Mon, 2 Mar 2015 12:50:53 +0000
> Subject: [PATCH] Add 'gama' atom for .mov only, reuses pngenc.c gamma values by sharing
>  function in new color_utils.[ch] file.
> 
> Use lrint() instead of round()

moving the code to a common place and adding gama atom support to mov
should be 2 seperate patches
also libavutils minor version should be bumped when new API is
added to libavutil

[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Freedom in capitalist society always remains about the same as it was in
ancient Greek republics: Freedom for slave owners. -- Vladimir Lenin
-------------- 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/20150303/27c4fb93/attachment.asc>