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

Michael Niedermayer michaelni at gmx.at
Fri Sep 20 00:28:32 CEST 2013


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 ?

the change is probably ok but the commit message is missing a
reference to upon what this is based

[...]

thanks

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Complexity theory is the science of finding the exact solution to an
approximation. Benchmarking OTOH is finding an approximation of the exact
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20130920/7be9b57f/attachment.asc>


More information about the ffmpeg-devel mailing list