[vlc-devel] [PATCH] include: add enum for chroma location

[Rémi Denis-Courmont]

Le 2016-04-16 16:10, Jan Ekström a écrit :
> This lets us handle chroma location correctly further down the
> rendering chain (even if in general things can be assumed by just
> the video track's format).
> ---
>  include/vlc_es.h | 11 +++++++++++
>  1 file changed, 11 insertions(+)
>
> diff --git a/include/vlc_es.h b/include/vlc_es.h
> index 9b582cc..471d2b3 100644
> --- a/include/vlc_es.h
> +++ b/include/vlc_es.h
> @@ -235,6 +235,16 @@ typedef enum video_color_space_t
>  } video_color_space_t;
>
>  /**
> + * Video chroma location
> + */
> +typedef enum video_chroma_location_t
> +{
> +    CHROMA_LOCATION_UNDEF,
> +    CHROMA_LOCATION_LEFT,   /*< Most common in MPEG-2 Video, 
> H.264/265 */
> +    CHROMA_LOCATION_CENTER, /*< Most common in MPEG-1 Video, JPEG */
> +} video_chroma_location_t;
> +
> +/**
>   * video format description
>   */
>  struct video_format_t
> @@ -265,6 +275,7 @@ struct video_format_t
>      video_color_primaries_t primaries;                  /**< color
> primaries */
>      video_transfer_func_t transfer;                   /**< transfer
> function */
>      video_color_space_t space;                        /**< YCbCr
> color space */
> +    video_chroma_location_t chroma_location;          /**< YCbCr
> chroma location */
>  };

Nit: 79 columns.

Consider adding a patch for src/input/es_out.c meta too.

>
>  /**

-- 
Rémi Denis-Courmont
http://www.remlab.net/