[FFmpeg-devel] [PATCH] lavfi/color: use AVOptions

[Stefano Sabatini]

On date Tuesday 2012-06-19 19:28:36 +0000, Paul B Mahol encoded:
> On 6/19/12, Stefano Sabatini <stefasab at gmail.com> wrote:
> > On date Tuesday 2012-06-19 16:07:45 +0000, Paul B Mahol encoded:
[...]
> > I see code duplication but removing it seems not worth the effort,
> > hopefully we'll get rid of it when we'll drop the old syntax (next
> > bump?).
> 
> Just lets not forget to remove it.

The best way is to use #ifdef trickery and automatically remove the
code at the next major bump.

> > [...]
> >
> > Looks fine otherwise if tested, thanks.
> 
> Tested, could not find anything obviously wrong. Applied.

Forgot to say, bump micro in these cases (feature extensions or option
addition), so we can know if the feature is supported given the
version number (but IMO no need to bump it now).
-- 
FFmpeg = Fast and Furious Multipurpose Peaceful Erotic Gymnast