[FFmpeg-devel] [PATCH] avfilter: add skipblend filter

Matthias Troffaes matthias.troffaes at gmail.com
Tue Jun 6 12:09:04 EEST 2017


Dear Moritz,

On Mon, Jun 5, 2017 at 3:21 PM, Moritz Barsnick <barsnick at gmx.net> wrote:
> I can't comment on the rest (and still really like the concept), but
> just this:
>
>> +Allowed values are positive integers between @code{1} and @code{65535}.
>
> This maximum value is no longer correct.

It's correct as far as I can tell. From the code:

{ "step", "set frame step",  OFFSET(frame_step), AV_OPT_TYPE_INT64,
{.i64=1}, 1, UINT16_MAX, FLAGS},

So you can see that UINT16_MAX (=65535) is the maximum value for the
step size, as documented. (Note that the type is AV_OPT_TYPE_INT64
because UINT16_MAX doesn't necessarily fit into an int, as you
correctly pointed out earlier.)

>> +Allowed values are positive integers between @code{1} and @code{step},
>> +where @code{1} corresponds to no motion blur, and @code{step}
>> +corresponds to maximal motion blur.
>
> Just wondering: Isn't this also useful for a slideshow-like
> transition/fade, not just for motion blur? (I'm saying: If so, the user
> needs to know.)

Good point, one could use it that way too, yes. I don't think it's
essential to the patch, but I'd be happy to submit a follow-on patch
to mention this use as well.

> I think I need to build and test it, to see if it fits
> my needs. ;)

Ok, thanks!

Kind regards,
Matthias


More information about the ffmpeg-devel mailing list