On Mon, Aug 29, 2016 at 5:53 PM, Ronald S. Bultje <rsbul...@gmail.com> wrote:
> Hi, > > On Mon, Aug 29, 2016 at 11:23 AM, Vittorio Giovara < > vittorio.giov...@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.giov...@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.) > Still breaks backward compatibility. _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
