lavc: do not leak frames in avcodec_decode_audio4

Notwithstanding the return value the frame should be freed if
is not going to be used.

Reported-by: Matthew Wolenetz <wolenetz@chromium.org>

nut: use meaningful error values

Fix the mispresented EINVAL for EOF on partial files, among the other.

FATE: use a less ambiguous end time for filter-trim-time test

0.1 seconds at 25 fps is exactly 2.5 frames, which may be rounded either
to 2 or 3 on different platforms.

configure: add dependencies to lavfi .pc file

The scale filter depends on sws, resample on lavr, and movie on lavf and
lavc.

Fixes Bug 502.

configure: make avconv select (a)trim filters

They are now used for -ss/-t

avconv: remove -deinterlace

It is incompatible with refcounted frames and since it's been deprecated
for a long time now, fixing it is not worth the effort.

indeo3: use unaligned reads on reference blocks.

They are not guaranteed to be aligned.
Fixes Bug 503.

CC:libav-stable@libav.org

jpeg2000dec: initialize static data just once

jpeg2000dec: cleanup properly after decoding errors

Fixes memory leak in fate-redcode-demux.

fate-jpeg2000-dcinema: force output to xyz12le

Fixes Fate tests on big-endian configs.

vf_scale: add endianness conversion pixel format in query_formats

swscale: support endianness conversion for AV_PIX_FMT_XYZ12

swscale: add support for endianness only conversion

Use bitfields in FormatEntry array to avoid wasting an int for each flag.

pixdesc: add function to return pixel format with swapped endiannes

swscale: Use alpha from the right row in yuva2rgba_c

Every other pixel had the alpha channel taken from the wrong
row.

This fixes bug 504.

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

swscale: Remove unnecessary backslashes

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

jpeg2000: Fix uninitialized data errors pointed out by valgrind/memcheck

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

x86: hpeldsp: Move avg_pixels8_x2_mmx() out of hpeldsp_rnd_template.c

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

x86: hpeldsp: Only compile MMX hpeldsp code if MMX is enabled

x86: More specific ifdefs for dsputil/hpeldsp init functions

wav: Always seek to an even offset

RIFF chunks are aligned to 16bit according to the specification.

Bug-Id:500
CC:libav-stable@libav.org

nut: use a define for the nut version

Ease switching version in the future and make evident why that value.

bgmc: Do not mark ff_bgmc_decode_init() as av_cold

The function is possibly called once per block.

avcodec: Add av_cold attributes to end functions missing them

avformat: Add av_cold attributes to init functions missing them

configure: Check for getenv

When targeting the "windows store application" (metro) API subset
(or the windows phone API subset), the getenv function isn't
available. If it is unavailable, just define getenv to NULL.

The check uses check_func_headers, since the function actually
might exist in the libraries, but is hidden in the headers.

The fallback is in config.h since msvc can't do -D defines with
parameters on the command line, and it's used both within the
libraries and the frontend applications (so a libavutil internal
header wouldn't be enough).

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

swscale: Add av_cold attributes to init functions missing them

avutil: Add av_cold attributes to init functions missing them

lavfi: let gcc realign the stack on public graph driving functions

The functions which actually drive the filter graph by pushing
frames through it need to ensure an aligned stack for SIMD functions.

This fixes a crash in YADIF filter when using a mingw build in a MSVC
application.

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

sctp: Add a note on why sctp_recvmsg and sctp_sendmsg have ff_-prefixes

avs: Increase magic value score above file extension score

This should fix misdetection of AVS files as AviSynth scripts
when AviSynth support is enabled (Bugzilla #357).

avformat: Add AVPROBE_SCORE_EXTENSION define and use where appropriate

build: output-example: Add avutil to ELIBS in link command

output-example links against swscale, which depends on avutil.  In standard
configurations, by pure good luck, the libs before swscale pull in all of
swscale's dependencies and linking succeeds.  However, in some configurations
like --disable-asm this is not the case and linking fails.

Hardcode the dependency to avoid a more general Makefile refactoring.

avresample: Add av_cold attributes to init functions missing them

avfilter: Add av_cold attributes to init/uninit functions

avcodec: Add av_cold attributes to init functions missing them

nut: K&R formatting cosmetics

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

configure: Add basic valgrind-memcheck support

With the parameter --valgrind-memcheck, the configure script sets
reasonable defaults that can be overridden as explained in the
documentation.

The idea of using set_defaults is from Luca Barbato.

build: normalize coverage.info

Without this, lcov sometimes misses to normalize paths that contain "/./".
Also, ignore uninteresting hits in system headers.

build: tune down the output of lcov/gcov

In the default settings, both tools produce a lot of unhelpful noise.

wav: return meaningful errors

silly typo fixes

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