Signed-off-by: NReimar Döffinger <Reimar.Doeffinger@gmx.de>

Signed-off-by: NReimar Döffinger <Reimar.Doeffinger@gmx.de>

Signed-off-by: NMichael Niedermayer <michaelni@gmx.at>

Signed-off-by: NMichael Niedermayer <michaelni@gmx.at>

Signed-off-by: NMichael Niedermayer <michaelni@gmx.at>

This fixes pointer type warnings
The context should have been unused so this should not fix any real bug
Signed-off-by: NMichael Niedermayer <michaelni@gmx.at>

Fixes part of Ticket3466
Found-by: Andrey_Karpov / PVS-Studio
Signed-off-by: NMichael Niedermayer <michaelni@gmx.at>

Signed-off-by: NMichael Niedermayer <michaelni@gmx.at>

The old row index looks weird and wrong to me
Signed-off-by: NMichael Niedermayer <michaelni@gmx.at>

Fixes crash on SSE2
Fixes Ticket2443

Note, its possible to restructure the code to avoid the 16pixel limitation,
and such change is welcome!
Signed-off-by: NMichael Niedermayer <michaelni@gmx.at>

Signed-off-by: NMichael Niedermayer <michaelni@gmx.at>

Replace literal numbers by named R_MAX and correct their value

Fixes half of Ticket2443

Based on patch by: João Bernardo <jbvsmo@gmail.com>
Signed-off-by: NMichael Niedermayer <michaelni@gmx.at>

dsputil_init() is deprecated and not meant to be exported.
ff_dsputil_init() is internal to libavcodec and thus can not be used.
avpriv_dsputil_init() is the version shared between libraries.

This commit fixes 3 unjustified libavfilter deprecated warnings.

This is mostly automated global search and replace

The deprecated aconvert filter is disabled, if it still has users
it should be updated
Signed-off-by: NMichael Niedermayer <michaelni@gmx.at>

Fixes compilation with clang on netbsd with --disable-optimizations.

Tested-by: Paul B Mahol

Signed-off-by: NMichael Niedermayer <michaelni@gmx.at>

Signed-off-by: NMichael Niedermayer <michaelni@gmx.at>

Signed-off-by: NPaul B Mahol <onemda@gmail.com>

This is following 568c70e7.

Signed-off-by: NMichael Niedermayer <michaelni@gmx.at>

Fix a segfault.

Reduce log clutter, consistent with 1a49a169.

Add NULL check on avctx before calling avcodec_close() on it.

Fix trac ticket #1472.

Signed-off-by: NPaul B Mahol <onemda@gmail.com>

Signed-off-by: NMichael Niedermayer <michaelni@gmx.at>

Sometimes the scan finds nothing that qualifies for addition to
the array and pos is zero after the loops.  The code forces pos to
1 and the array is then processed as if it had one valid element in it,
producing some amusing but not very useful results.

I don't see the rationale for this.  If pos is zero coming out of the
loops, the only appropriate thing to do is set t->angle to zero.  The
attached patch does that.  It's worked properly in several tests so far.
Signed-off-by: NMichael Niedermayer <michaelni@gmx.at>

Variable totalangles was created and assigned, but never used.
Signed-off-by: NRay Simard <rhs.ffmpeg@sylvan-glade.com>
Signed-off-by: NStefano Sabatini <stefasab@gmail.com>

Initialize Transform structs t and orig to zero.
Signed-off-by: NRay Simard <rhs.ffmpeg@sylvan-glade.com>
Signed-off-by: NStefano Sabatini <stefasab@gmail.com>

Using the double variant causes several pointless conversions between
double and int.
Worse, one of the conversions is in an inner loop together with a
function using MMX, resulting in undefined behaviour.
Based on debugging by Ray Simard.
Signed-off-by: NReimar Döffinger <Reimar.Doeffinger@gmx.de>
Tested-by: NRay Simard <rhs.ffmpeg@sylvan-glade.com>