This avoids recalculating the value over and over again.

Drop non-informative _bfin suffix, rename some files for consistency with
other architectures, rename others to reflect their content.

This saves some space in configurations that do not enable hpeldsp.

The functions it contains are used from unconditionally compiled code.

This is done similarly for all other architectures.

Also add a few #endif comments.

They were marked as broken and disabled in 2009 and will never get fixed.

They were marked as broken and disabled in 2009 and will never get fixed.

Also switch from "tbl" to "tab" name suffixes.

This fixes a leftover from e44d7c65.

It has been disabled as 'broken' over 3 years ago in b716a792

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

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

This way, the special IDCT permutations are no longer needed. This
is similar to how H264 does it, and removes the dsputil dependency
imposed by the scantable code.

Also remove the unused type == 0 cases from the plain C version
of the idct.
Signed-off-by: Martin Storsjö <martin@martin.st>

While this change isn't bitexact, the IDCTs weren't bitexact to
start with either.

This simplifies decoupling vp3 from dsputil.
Signed-off-by: Martin Storsjö <martin@martin.st>

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

There's no point in these pointers not being const.
Signed-off-by: Martin Storsjö <martin@martin.st>

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

In the non-bitexact mode, vp3 currently decodes to the same
frame crcs as before 28f9ab70 (and the output visually looks
correct).
Signed-off-by: Martin Storsjö <martin@martin.st>

This way, the special IDCT permutations are no longer needed. Bfin code
is disabled until someone updates it. This is similar to how H264 does
it, and removes the dsputil dependency imposed by the scantable code.
Signed-off-by: Michael Niedermayer <michaelni@gmx.at>

This avoids SIMD-optimized functions having to sign-extend their
line size argument manually to be able to do pointer arithmetic.

Added a missing header file.

The block must be set to 0.

It does not help as an abstraction and adds dsputil dependencies.
Signed-off-by: Ronald S. Bultje <rsbultje@gmail.com>

This allows us to get rid of them on the next major bump. All of the
above are functionally irrelevant, and most of them are unused, except
the vp3 one, which is used wrongly in the bfin arch optimizations.

Reveiwed-by: Stefano Sabatini <stefasab@gmail.com>
Signed-off-by: Michael Niedermayer <michaelni@gmx.at>