[FFmpeg-devel] [PATCHv2] vf_colorspace: Allow overriding input color properties

[Ronald S. Bultje]

Hi,

On Mon, Aug 29, 2016 at 11:23 AM, Vittorio Giovara <
vittorio.giovara at gmail.com> wrote:

> The filter needs input frames with color properties filled out by
> the decoder. Since this is not always possible, add input options to
> the filter so that user may override color space, color primaries,
> transfer characteristics, and color range, as well as a generic option
> to set all properties at once.
>
> Signed-off-by: Vittorio Giovara <vittorio.giovara at gmail.com>
> ---
> * added iall option
> * updated filter documentation
>
> Please CC.
> Vittorio
>
>  doc/filters.texi            | 20 ++++++++++++++++++++
>  libavfilter/vf_colorspace.c | 39 ++++++++++++++++++++++++++++++++++-----
>  2 files changed, 54 insertions(+), 5 deletions(-)


lgtm.

(I wonder if the error message - if the input AVFrame has no
trc/rng/csp/prm - should be changed to reflect that you can override them
using the added properties? This isn't a big deal but maybe someone feels
that's important.)

Ronald