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

[Ronald S. Bultje]

Hi,

On Sun, Oct 30, 2016 at 9:14 AM, Ronald S. Bultje <rsbultje at gmail.com>
wrote:

> 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.
>

Pushed.

Ronald