sbrdsp: Unroll and use integer operations

This patch can be controversial, by assuming floats are IEEE-754 and
particular behaviour of the FPU will get in the way.
Timing on Arrandale and Win32 (thus, x87 FPU is used in the reference).

sbr_qmf_pre_shuffle_c: 115 to 76
sbr_neg_odd_64_c: 84 to 55
sbr_qmf_post_shuffle_c: 112 to 83

Signed-off-by: Diego Biurrun <diego@biurrun.de>

sbrdsp: Unroll sbr_autocorrelate_c

1410 cycles to 1148 on Arrandale/Win64

Signed-off-by: Diego Biurrun <diego@biurrun.de>

x86: sbrdsp: Implement SSE2 qmf_deint_bfly

Sandybridge: 47 cycles

Having a loop counter is a 7 cycle gain.
Unrolling is another 7 cycle gain.
Working in reverse scan is another 6 cycles.

Signed-off-by: Diego Biurrun <diego@biurrun.de>

compat: msvc: Make sure the object files are included when linking statically

If building libav with -MD in the cflags (for making the MSVC compiler
generate code for using a dynamically linked libc), the system headers
that declare strtod, snprintf and vsnprintf declare the functions as
imported from a DLL. To hook up wrappers of our own for these functions,
the function names are defined to avpriv_*, so that the calling code
within libav calls the wrappers instead. Since these functions
are declared to be imported from DLLs, the calling code expects to
load them from DLL import function pointers (creating references to
_imp__avpriv_strtod instead of directly to avpriv_strtod). If the
libav libraries are not built as DLLs, no such function pointers (as
the calling code expects) are created.

The linker can fix this up automatically in some cases (producing
warnings LNK4217 and LNK4049), if the object files are already
included. By telling the linker to try to include those symbols
(without the _imp prefix as the calling code ends up using),
we get the object files included, so that the linker can do the
automatic fixup. This is done via config.h, so that all (or at least
most) of the object files in our libraries force including the compat
files, to make sure they are included regardless of what files from our
static libraries actually are included.

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

id3v2: check for end of file while unescaping tags

Prevent a serious out of buffer bound write.

Reported-by: Mateusz "j00ru" Jurczyk and Gynvael Coldwind
CC:libav-stable@libav.org

avconv: do not send non-monotonous DTS to the muxers.

Hack partially based on a commit by Michael Niedermayer <michaelni@gmx.at>
Should fix (or work around) bug 458.

avconv: improve -re implementation

Integrate the code in the packet reading function, instead of inserting
sleeps in many places.
This is simpler to follow and should work better.

matroskaenc: add an option to put the index at the start of the file

matroskaenc: cosmetics, reorder the #includes

vc1_parser: Set field_order.

Signed-off-by: Anton Khirnov <anton@khirnov.net>

mpegvideo_parser: Set field_order.

Signed-off-by: Anton Khirnov <anton@khirnov.net>

avcodec: Add field order information to AVCodecParserContext.

Signed-off-by: Anton Khirnov <anton@khirnov.net>

x86: float dsp: butterflies_float SSE

97c -> 49c
Some codecs could benefit from more unrolling, but AAC doesn't.

lavf-regression: use -frames instead of -t for image tests

-t 0.5 is 12.5 frames at 25 fps, which may round to either 12 or 13 on
different platforms.

lavc: schedule AVCodecContext.lowres for removal on next bump.

It has been deprecated some time ago, but was forgotten during the last
bump.

configure: Don't do enable_deep_weak on disabled variables

This avoids cases where configure tries to weakly enable an item
which actually is disabled, ending up still enabling dependencies
of the item which itself is only enabled weakly.

More concretely, the h264 decoder suggests error resilience, which
is then enabled weakly (unless manually disabled). Previously,
dsputil, which is a dependency of error resilience, was enabled
even if error resilience wasn't enabled in the end.

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

configure: Use a different variable name in push/popvar

The variable name 'var' is commonly used to iterate through arguments
in other functions. When the pushvar function internally uses the
variable 'var', it makes pushing/popping the variable 'var' not
work as intended.

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

wav: K&R formatting cosmetics

Signed-off-by: Diego Biurrun <diego@biurrun.de>

aac: Remove aacpsy.h header, unused since 78e65cd7726

cavs: Remove cavsdata.h header, unused since 88386feefd

x86: dsputil: Move cavs and vc1-specific functions where they belong

x86: dsputil: Move avg_pixels16_mmx() out of rnd_template.c

The function does not do any rounding, so there is no point in
keeping it in a round template file.

x86: dsputil: Move avg_pixels8_mmx() out of rnd_template.c

The function is only instantiated once, so there is no point
in keeping it in a template file.

x86: Move duplicated put_pixels{8|16}_mmx functions into their own file

id3v2: K&R formatting cosmetics

Signed-off-by: Diego Biurrun <diego@biurrun.de>

ppc: Only compile AltiVec FFT assembly when AltiVec is enabled

avutil: doxy: Small clarification for av_buffer_create()

Explicitly saying it can't hurt.

Signed-off-by: Derek Buitenhuis <derek.buitenhuis@gmail.com>

avcodec: doxy: Clarify how to use buf[] when implementing get_buffer2()

Signed-off-by: Derek Buitenhuis <derek.buitenhuis@gmail.com>

avcodec: doxy: Clarify what get_buffer2 must fill in buf[]

Signed-off-by: Derek Buitenhuis <derek.buitenhuis@gmail.com>

avformat: Drop unnecessary ff_ name prefixes from static functions

ppc: Drop unnecessary ff_ name prefixes from static functions

x86: Drop unnecessary ff_ name prefixes from static functions

arm: Drop unnecessary ff_ name prefixes from static functions

avcodec: Drop unnecessary ff_ name prefixes from static functions

ppc: More consistent arch initialization

fft: Remove pointless #ifdefs around function declarations

ppc: Move AltiVec utility headers out of AltiVec ifdefs

Now that the headers themselves have ifdef protection this is no
longer necessary and more consistent with normal include handling.

ppc: util_altivec: Surround AltiVec-related code by appropriate ifdefs

This prevents non-AltiVec-enabled compilers from choking.

ppc: More consistent names for H.264 optimizations files

mpegaudiosp: More consistent names for ppc/x86 optimization files

avconv: make output -ss insert trim/atrim filters.

This makes output -ss sample-accurate for audio and will allow further
simplication in the future.

avconv: make -t insert trim/atrim filters.

This makes -t sample-accurate for audio and will allow further
simplication in the future.

Most of the FATE changes are due to audio now being sample accurate. In
some cases a video frame was incorrectly passed with the old code, while
its was over the limit.

avconv_opt: allocate an OutputFile before the streams.

This way OutputFile variables like recording time can be used when
creating the streams.

lavfi: add trim and atrim filters.

lavfi: add an asetpts filter

vf_setpts: cosmetics, reformat

graphparser: only print filter arguments if they are non-NULL

af_channelmap: ensure the output channel layout is valid.

af_channelmap: remove now unnecessary goto

Options are freed from the generic code now, there is no need to call
av_opt_free() from the filter.

vf_split: fix description

It now allows an arbitrary number of inputs, not just two.

lavc: clear AVBuffers on decoded frames if refcounted_frames is not set

Otherwise some code might mistakenly think it is allowed to unref those
buffers, which would lead to double unref.

FATE: add an additional indeo3 test

It is a part of the sample that got broken by recent indeo3 changes.

indeo3: use put_pixels instead of put_no_rnd_pixels in copy_cell()

They are the same in this case, except put_pixels also has blocksizes
smaller than 8.

indeo3: fix off by one in MV validity check

CC:libav-stable@libav.org

x86: dsputil: Remove a set of pointless #ifs around function declarations

x86: dsputil: cosmetics: Group ff_{avg|put}_pixels16_mmxext() declarations

x86: hpeldsp: Remove unused macro definitions

aac: check the maximum number of channels

Broken bitstreams could report a larger than specified number of
channels and cause outbound writes.

CC:libav-stable@libav.org

h264_refs: Do not print check_opcodes() return value

The return value provides no useful information and removing the printing
avoids the following warning:
libavcodec/h264_refs.c:788:15: warning: 'i' may be used uninitialized in this function [-Wuninitialized]

fate: Invoke pixfmts lavfi tests through fate-run.sh

fate: Invoke pixdesc lavfi tests through fate-run.sh

fate: Invoke standard lavfi tests through fate-run.sh

oma: K&R formatting cosmetics

avplay: remove a warning

Set the data field in the flush_pkt to the pointer to the actual packet.
The field needs to contain a valid unique pointer, no read nor writes
are ever made to it.

hls, segment: fix splitting for audio-only streams.

CC:libav-stable@libav.org

x86: ac3dsp: Remove 3dnow version of ff_ac3_extract_exponents

The function requires increasing the fuzz factor for the ac3/eac3 encode
tests and even so makes fate fail. It only provides a slight encoding
speedup for legacy CPUs that do not support SS2. Thus its benefit is not
worth the trouble it creates and fixing it would be a waste of time.

x86: Rename dsputil_rnd_template.c to rnd_template.c

This makes it less confusing when this template is shared both by
dsputil and by hpeldsp.

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

lavf: introduce AVFMT_TS_NEGATIVE

Most formats do not support negative timestamps, shift them to avoid
unexpected behaviour and a number of bad crashes.

CC:libav-stable@libav.org

Signed-off-by: Anton Khirnov <anton@khirnov.net>
Signed-off-by: Luca Barbato <lu_zero@gentoo.org>

fate: add CVFC1_Sony_C to h264 conformance tests

The sample is already included in the FATE suite, but is not tested
because cropping wasn't fully supported before.

Signed-off-by: Anton Khirnov <anton@khirnov.net>

doc/APIchanges: add missing hashes and dates

x86: Get rid of duplication between *_rnd_template.c

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

mpegvideo: unref cur/next/prev frames when flushing

Signed-off-by: Anton Khirnov <anton@khirnov.net>

x86: Factorize duplicated inline assembly snippets

Signed-off-by: Diego Biurrun <diego@biurrun.de>

jpeg2000: Add mutlti-threading support to decoder

Signed-off-by: Diego Biurrun <diego@biurrun.de>

x86: Move some conditional code around to avoid unused variable warnings

jpeg2kdec: output is native endian AV_PIX_FMT_XYZ12

threads: always call thread_finish_setup for intra codecs

Intra codecs do not need an update_thread_context() function and never
call ff_thread_finish_setup(). They rely on ff_thread_get_buffer()
calling it. So call it even if the get_buffer2 function pointer is
avcodec_default_get_buffer2 and it has not been called before.

avcodec: Bump minor for JPEG 2000 decoder

JPEG 2000 decoder for DCinema

Based on the 2007 GSoC project from Kamil Nowosad <k.nowosad@students.mimuw.edu.pl>
Updated to current programming standards, style and many more small
fixes by Diego Biurrun <diego@biurrun.de>.

Signed-off-by: Diego Biurrun <diego@biurrun.de>

sh4: Remove dubious aligned dsputil code

The code represents a considerable maintenance burden and it is not
clear that it gives a noticeable benefit to outweigh this after 10
years of improvements in compiler technology since its creation.

x86: cavs: Refactor duplicate dspfunc macro

mxfenc: Use correct printf format specifier for int64_t

libavformat/mxfenc.c:1861:9: warning: format '%lld' expects argument of type 'long long int', but argument 5 has type 'int64_t' [-Wformat]

h264: Drop unused variable

x86: cavs: Put mmx-specific code into its own init function

Before, this code was labeled as mmxext and enabled both for the
3dnow and the mmxext case.

x86: Remove some duplicate function declarations

afifo: fix request_samples on the last frame in certain cases

The current code can fail to return the last frame if it contains
exactly the requested number of samples.

Fixes the join filter test, which previously did not include the last
408 samples in most cases.

CC:libav-stable@libav.org

Signed-off-by: Diego Biurrun <diego@biurrun.de>

riff: Factor out WAVEFORMATEX parsing

Makes the code simpler to follow.

Signed-off-by: Diego Biurrun <diego@biurrun.de>

riff: Add ACTRAC3+ guid

Signed-off-by: Luca Barbato <lu_zero@gentoo.org>
Signed-off-by: Diego Biurrun <diego@biurrun.de>

riff: Perform full lookup on WAVEFORMATEXTENSIBLE subformat guid

Signed-off-by: Luca Barbato <lu_zero@gentoo.org>
Signed-off-by: Diego Biurrun <diego@biurrun.de>

riff: Move guid structs and helper functions into riff

Signed-off-by: Luca Barbato <lu_zero@gentoo.org>
Signed-off-by: Diego Biurrun <diego@biurrun.de>

riff: check for eof if chunk size and code are 0

Prevent an infinite loop.

Inspired by a patch from Michael Niedermayer

CC: libav-stable@libav.org
Signed-off-by: Diego Biurrun <diego@biurrun.de>

riff: K&R formatting cosmetics

Signed-off-by: Diego Biurrun <diego@biurrun.de>

ppc: hpeldsp: Include attributes.h

This fixes building in configurations where altivec is disabled.

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

x86: Remove unused inline asm instruction defines

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

vc1: Remove now unused variables

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

cosmetics: bfin: Fix indentation in the dsputil init function

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

dsputil: Remove non-8bpp draw_edge

It is never used.

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

dsputil: Merge 9-10 bpp functions for get_pixels and draw_edge

These only care about pixel storage unit size, not actual bits
used (i.e. they don't clip).

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

dsputil: Remove unused 32-bit functions

Previously, if dct_bits was set to 32, we used separate 32-bit
versions of these functions. Since dct_bits now is removed,
remove the unused 32-bit versions of the functions.

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

dsputil: Remove dct_bits

dct_bits is never set except in h264, where it is never used,
thus remove it.

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