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 michae...@gmx.at 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

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 kevin.j.wheat...@gmail.com Date: Mon, 2 Mar 2015 12:50:53 + Subject: [PATCH] Add 'gama' atom for .mov only, reuses

[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 g...@nand.wakku.to 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

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 g...@nand.wakku.to wrote: +static int png_get_gama(enum AVColorTransferCharacteristic trc, uint8_t *buf) +{ +double gamma; +switch (trc) { +case