[FFmpeg-devel] [PATCH] fraps: use BT.709 colorspace for YUV Fraps versions

[Hendrik Leppkes]

On Fri, Sep 20, 2013 at 12:28 AM, Michael Niedermayer <michaelni at gmx.at> wrote:
> On Tue, Sep 17, 2013 at 06:26:43PM +0000, Paul B Mahol wrote:
>> From: Hendrik Leppkes <h.leppkes at gmail.com>
>>
>> ---
>>  libavcodec/fraps.c | 1 +
>>  1 file changed, 1 insertion(+)
>>
>> diff --git a/libavcodec/fraps.c b/libavcodec/fraps.c
>> index a1d5a17..70c1104 100644
>> --- a/libavcodec/fraps.c
>> +++ b/libavcodec/fraps.c
>> @@ -201,6 +201,7 @@ static int decode_frame(AVCodecContext *avctx,
>>
>>      avctx->pix_fmt = version & 1 ? AV_PIX_FMT_BGR24 : AV_PIX_FMT_YUVJ420P;
>>      avctx->color_range = version & 1 ? AVCOL_RANGE_UNSPECIFIED : AVCOL_RANGE_JPEG;
>> +    avctx->colorspace = version & 1 ? AVCOL_SPC_UNSPECIFIED : AVCOL_SPC_BT709;
>
> based on what is this ?
> some specification? comparission with some decoder ? encoder ?
>

Its based both on comparison to the reference decoder, and comments
from the fraps developers upon inquiry how to handle the yuv fraps
variants.

- Hendrik