Re: [FFmpeg-devel] Gamma function (was Re: [PATCH] lavc/pngenc: Support writing colorspace tags.)

2015-03-02 Thread Kevin Wheatley
Something like this - note this adds no tests, but fate still passes for me. Kevin From db02ae26c3c4278da4ed328e767bab98271da51e Mon Sep 17 00:00:00 2001 From: Kevin Wheatley Date: Mon, 2 Mar 2015 12:50:53 + Subject: [PATCH] Add 'gama' atom for .mov only, reuses pngenc.c gamma values by shari

Re: [FFmpeg-devel] Gamma function (was Re: [PATCH] lavc/pngenc: Support writing colorspace tags.)

2015-03-02 Thread Kevin Wheatley
On Mon, Mar 2, 2015 at 12:17 PM, Michael Niedermayer wrote: > the various colorspace options should pass from decoder over the > video filter chain to the encoder and then muxer > its best to change them from the command line through a video > filter, this also ensures that the value reaching the

Re: [FFmpeg-devel] Gamma function (was Re: [PATCH] lavc/pngenc: Support writing colorspace tags.)

2015-03-02 Thread Michael Niedermayer
On Mon, Mar 02, 2015 at 11:00:43AM +, Kevin Wheatley wrote: > On Sat, Feb 28, 2015 at 1:50 AM, Niklas Haas wrote: > > > +static int png_get_gama(enum AVColorTransferCharacteristic trc, uint8_t > > *buf) > > +{ > > +double gamma; > > +switch (trc) { > > +case AVCOL_TRC_BT709:

[FFmpeg-devel] Gamma function (was Re: [PATCH] lavc/pngenc: Support writing colorspace tags.)

2015-03-02 Thread Kevin Wheatley
On Sat, Feb 28, 2015 at 1:50 AM, Niklas Haas wrote: > +static int png_get_gama(enum AVColorTransferCharacteristic trc, uint8_t *buf) > +{ > +double gamma; > +switch (trc) { > +case AVCOL_TRC_BT709: > +case AVCOL_TRC_SMPTE170M: > +case AVCOL_TRC_SMPTE240M: > +