[vlc-devel] [PATCH 3/3] vpx: Read color space information

[Jean-Baptiste Kempf]

LGTM.

170 and 240 are the same?

On 25 Sep, Vittorio Giovara wrote :
> ---
>  modules/codec/vpx.c | 18 ++++++++++++++++++
>  1 file changed, 18 insertions(+)
> 
> diff --git a/modules/codec/vpx.c b/modules/codec/vpx.c
> index d0dd57f..c681de6 100644
> --- a/modules/codec/vpx.c
> +++ b/modules/codec/vpx.c
> @@ -239,6 +239,24 @@ static picture_t *Decode(decoder_t *dec, block_t **pp_block)
>          dec->fmt_out.video.i_sar_den = 1;
>      }
>  
> +    switch( img->cs )
> +    {
> +        case VPX_CS_SRGB:
> +        case VPX_CS_BT_709:
> +            v->space = COLOR_SPACE_BT709;
> +            break;
> +        case VPX_CS_BT_601:
> +        case VPX_CS_SMPTE_170:
> +        case VPX_CS_SMPTE_240:
> +            v->space = COLOR_SPACE_BT601;
> +            break;
> +        case VPX_CS_BT_2020:
> +            v->space = COLOR_SPACE_BT2020;
> +            break;
> +        default:
> +            break;
> +    }
> +
>      picture_t *pic = decoder_NewPicture(dec);
>      if (!pic)
>          return NULL;
> -- 
> 2.10.0
> 
> _______________________________________________
> vlc-devel mailing list
> To unsubscribe or modify your subscription options:
> https://mailman.videolan.org/listinfo/vlc-devel

-- 
With my kindest regards,

-- 
Jean-Baptiste Kempf
http://www.jbkempf.com/ - +33 672 704 734
Sent from my Electronic Device