[vlc-devel] [PATCH] include: split the color range from the color space with a boolean

[Vittorio Giovara]

On Sun, Apr 17, 2016 at 4:05 AM, Steve Lhomme <robux4 at videolabs.io> wrote:
> --
> replaces https://patches.videolan.org/patch/12992/
> ---
>  include/vlc_es.h | 12 ++++--------
>  1 file changed, 4 insertions(+), 8 deletions(-)
>
> diff --git a/include/vlc_es.h b/include/vlc_es.h
> index 9b582cc..19eb0d7 100644
> --- a/include/vlc_es.h
> +++ b/include/vlc_es.h
> @@ -224,14 +224,9 @@ typedef enum video_transfer_func_t
>  typedef enum video_color_space_t
>  {
>      COLOR_SPACE_UNDEF,
> -    COLOR_SPACE_BT601_LIMITED,
> -#define COLOR_SPACE_BT601 COLOR_SPACE_BT601_LIMITED
> -    COLOR_SPACE_BT601_FULL,
> -    COLOR_SPACE_BT709_LIMITED,
> -#define COLOR_SPACE_BT709 COLOR_SPACE_BT709_LIMITED
> -    COLOR_SPACE_BT709_FULL,
> -    COLOR_SPACE_BT2020_LIMITED,
> -#define COLOR_SPACE_BT2020 COLOR_SPACE_BT2020_LIMITED
> +    COLOR_SPACE_BT601,
> +    COLOR_SPACE_BT709,
> +    COLOR_SPACE_BT2020,
>  } video_color_space_t;
>
>  /**
> @@ -265,6 +260,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 */
> +    bool color_range_full;                      /**< 0-255 instead of 16-235 */
>  };
>
>  /**
> --

yeah I think this work better
-- 
Vittorio