[FFmpeg-devel] [PATCH]lavc/libvpx: Fix high-bitrate colour spaces

[James Zern]

Hi,

On Thu, Feb 18, 2016 at 3:30 PM, Carl Eugen Hoyos <cehoyos at ag.or.at> wrote:
> Please comment, Carl Eugen
> diff --git a/libavcodec/libvpx.c b/libavcodec/libvpx.c
> index a60d186..1e7010b 100644
> --- a/libavcodec/libvpx.c
> +++ b/libavcodec/libvpx.c
> @@ -46,14 +46,14 @@ static const enum AVPixelFormat vp9_pix_fmts_highbd[] = {
>      AV_PIX_FMT_YUV422P,
>      AV_PIX_FMT_YUV440P,
>      AV_PIX_FMT_YUV444P,
> -    AV_PIX_FMT_YUV420P10LE,
> -    AV_PIX_FMT_YUV422P10LE,
> -    AV_PIX_FMT_YUV440P10LE,
> -    AV_PIX_FMT_YUV444P10LE,
> -    AV_PIX_FMT_YUV420P12LE,
> -    AV_PIX_FMT_YUV422P12LE,
> -    AV_PIX_FMT_YUV440P12LE,
> -    AV_PIX_FMT_YUV444P12LE,
> +    AV_PIX_FMT_YUV420P10,
> +    AV_PIX_FMT_YUV422P10,
> +    AV_PIX_FMT_YUV440P10,
> +    AV_PIX_FMT_YUV444P10,
> +    AV_PIX_FMT_YUV420P12,
> +    AV_PIX_FMT_YUV422P12,
> +    AV_PIX_FMT_YUV440P12,
> +    AV_PIX_FMT_YUV444P12,
>      AV_PIX_FMT_NONE
>  };
>  #endif
>

> diff --git a/libavcodec/libvpx.c b/libavcodec/libvpx.c
> index a60d186..1e7010b 100644
> --- a/libavcodec/libvpx.c
> +++ b/libavcodec/libvpx.c
> @@ -46,14 +46,14 @@ static const enum AVPixelFormat vp9_pix_fmts_highbd[] = {
>      AV_PIX_FMT_YUV422P,
>      AV_PIX_FMT_YUV440P,
>      AV_PIX_FMT_YUV444P,
> -    AV_PIX_FMT_YUV420P10LE,
> -    AV_PIX_FMT_YUV422P10LE,
> -    AV_PIX_FMT_YUV440P10LE,
> -    AV_PIX_FMT_YUV444P10LE,
> -    AV_PIX_FMT_YUV420P12LE,
> -    AV_PIX_FMT_YUV422P12LE,
> -    AV_PIX_FMT_YUV440P12LE,
> -    AV_PIX_FMT_YUV444P12LE,
> +    AV_PIX_FMT_YUV420P10,
> +    AV_PIX_FMT_YUV422P10,
> +    AV_PIX_FMT_YUV440P10,
> +    AV_PIX_FMT_YUV444P10,
> +    AV_PIX_FMT_YUV420P12,
> +    AV_PIX_FMT_YUV422P12,
> +    AV_PIX_FMT_YUV440P12,
> +    AV_PIX_FMT_YUV444P12,
>      AV_PIX_FMT_NONE
>  };
>  #endif
>

The decoder won't change when built on a big-endian platform, so I don't think
this is correct.