[FFmpeg-devel] [PATCH 2/3] vf_colorspace: Add support for ycgco color space

[Ronald S. Bultje]

Hi,

On Sun, Oct 30, 2016 at 3:07 AM, Vittorio Giovara <
vittorio.giovara at gmail.com> wrote:

> Signed-off-by: Vittorio Giovara <vittorio.giovara at gmail.com>
> ---
> This is a little hackish, not sure how to represent the matrix otherwise.


Haha :) I have to admit I hadn't put much thought into YCgCo yet. It's a
little hacky but I guess it's OK...

I think the primary reason some people - at some point in ancient history,
certainly not today - might have cared about YCgCo is because the
conversion between RGB and YCgCo is so trivial. For example, YCgCo to RGB
is two butterflies (an interleaved add/sub pair), which is incredibly
appealing from a performance point of view. But we're not using that
property here...

I'm not saying it's bad, and it's probably totally irrelevant, especially
given how insignificant YCgCo is anyway, but it's worth noting.

Ronald