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

[Michael Niedermayer]

On Fri, Sep 20, 2013 at 02:27:04PM +0200, Hendrik Leppkes wrote:
> 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.

patch ok then with a commit message containing this in some way

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

In a rich man's house there is no place to spit but his face.
-- Diogenes of Sinope
-------------- 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/53b392a5/attachment.asc>