When slice_h is rounded up due to chroma subsampling, there's
a risk that jobnr * slice_h exceeds frame->height.

Prior to a638e918, this wasn't an issue for the last slice
of a frame, since slice_end was set to frame->height for the last
slice.

a638e918 tried to fix the case where other slices than the
last one would exceed frame->height (which can happen where the
number of slices/threads is very large compared to the frame
height).

However, the fix in a638e918 instead broke other cases,
where slice_h * nb_threads < frame->height. Therefore, make
sure the last slice always ends at frame->height.

CC: libav-stable@libav.org
Signed-off-by: NMartin Storsjö <martin@martin.st>

CC: libav-stable@libav.org

This is shorter and consistent across filters.

Deprecate AVFilterBuffer/AVFilterBufferRef and everything related to it
and use AVFrame instead.

Any alleged performance benefits gained from the split are purely
mythological and do not justify added code complexity.

A number of compilers, for example those from TI and IBM, choke on
these initialisers.  The current style is also quite ugly.
Signed-off-by: NMans Rullgard <mans@mansr.com>

Signed-off-by: NMartin Storsjö <martin@martin.st>

It is not used in any filters currently and is inherently evil. If
passing binary data to filters is required in the future, it should be
done with some AVOptions-based system.

It will allow adding new fields to it without ABI breaks.

They are only useful inside filters and we don't allow user filters for
now.

It is only useful inside filters and we don't allow user filters for
now.

Those functions are only useful inside filters. It is better to not
support user filters until the API is more stable.

Fixes several "‘static’ is not at beginning of declaration" warnings.

draw_slice() checks that the fade factor is < 65536 and only
calculates the fade if so. But the fade factor is clipped in
end_frame() by av_clip_uint16() to 65535, so the fade is calculated
for every frame.  This patch alters the check so that it compares with
< 65535 (UINT16_MAX).
Signed-off-by: NAnton Khirnov <anton@khirnov.net>

draw_slice() checks that planes 0 and 1 of AVFilterBufferRef's data
are not NULL before manipulating planes 1 and 2. This patch makes the
check against planes 1 and 2. More senseful and possibly more robust.
Signed-off-by: NAnton Khirnov <anton@khirnov.net>

Port fade filter from libavfilter soc repo, with minor fixes by
Stefano.
Signed-off-by: NMichael Niedermayer <michaelni@gmx.at>
Signed-off-by: NAnton Khirnov <anton@khirnov.net>