lavu: remove disabled avoptions cruft

lavu: remove disabled FF_API_GET_BITS_PER_SAMPLE_FMT cruft

Revert "avutil: make some tables visible again"

This reverts commit def8c94699877d015154a03d43e2b8f479cd89a0.

After the major bump this is no longer necessary.

Revert "avutil: Add a copy of ff_sqrt_tab back into avutil to restore ABI compatibility"

This reverts commit d15c21e5fa3961f10026da1a3080a3aa3cf4cec9.

After the major bump this is no longer necessary.

lavu: bump major to 52

In addition to the recent tables mess, the AVOption defaults behavior
changed, so an old lavc used with a new lavu will get completely messed
up defaults.

lavu: postpone recent deprecations until the next major bump

APIchanges: update lavr bump date

avconv: only apply presets when we have an encoder.

Fixes a crash when using a preset with stream copy.

CC: libav-stable@libav.org

atrac3: replace a calculation with FFALIGN()

This allocates 4 bytes less than the previous code if avctx->block_align is
a multiple of 4, but the extra 4 bytes is not really needed.

atrac3: remove unused ATRAC3Context field, sample_rate

atrac3: use sizeof(variable) instead of sizeof(type)

atrac3: simplify MDCT window calculation

atrac3: initialize static tables in AVCodec.init_static_data()

atrac3: separate window initialization from IMDCT initialization

atrac3: move the 'frame_factor' field from ATRAC3Context to where it is used

atrac3: remove unused ATRAC3Context field, bit_rate

atrac3: move the 'samples_per_frame' field from ATRAC3Context to where it is used

atrac3: remove unused ATRAC3Context field, samples_per_channel

atrac3: use AVCodecContext.block_align instead of keeping a private copy

atrac3: move the 'delay' field from ATRAC3Context to where it is used

atrac3: move the 'version' field from ATRAC3Context to where it is used

atrac3: use AVCodecContext.channels instead of keeping a private copy

atrac3: simplify some loop indexing

atrac3: cosmetics: pretty-printing and renaming

also does some minor refactoring.

pcm: define AVCodec instances only for enabled codecs

This defines an AVCodec only if the corresponding CONFIG option is
enabled instead of using the broad CONFIG_ENCODERS/DECODERS.

Signed-off-by: Mans Rullgard <mans@mansr.com>

libxvid: remove useless doxy comments.

lavc: remove stats_out from the options table.

Since it is declared as a string AVOption, the generic freeing code
attempts to free it on codec close. Some codecs might have already freed
it elsewhere (or didn't even allocate it with av_malloc() in the first
place), so this might lead to an invalid free.

There is no point in having this field accessible as an AVOption, so
remove it from the options table.

Fixes Bug 380.

CC: libav-stable@libav.org

riff: remove a write-only variable

lavc: extend frame_size doxy.

alacdec: set bits_per_raw_sample

rtpdec: Don't pass a non-AVClass pointer as log context

The log context is assumed to start with an AVClass pointer.

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

rtsp: Update a comment to the current filename scheme

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

avcodec: handle AVERROR_EXPERIMENTAL

Error out on init if a codec with CODEC_CAP_EXPERIMENTAL is requested
and strict_std_compliance is not FF_COMPLIANCE_EXPERIMENTAL.

Move the check from avconv to avcodec_open2() and return
AVERROR_EXPERIMENTAL accordingly.

Signed-off-by: Luca Barbato <lu_zero@gentoo.org>

avutil: Add AVERROR_EXPERIMENTAL

Signed-off-by: Luca Barbato <lu_zero@gentoo.org>

avcodec: prefer decoders without CODEC_CAP_EXPERIMENTAL

Signed-off-by: Luca Barbato <lu_zero@gentoo.org>

fate: adpcm: cosmetics: Sort test entries

This also places tests of related formats together.

fate: adpcm: Add dependencies

svq3: cosmetics: Drop useless parentheses

svq3: K&R formatting cosmetics

fate: Introduce ENCMUX macro for tests that require encoders and a muxer

ffv1: Add missing #includes to header file

doc/platform: Add info on shared builds with MSVC

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

doc/platform: Move a caveat down to the notes section

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

ARM: reinstate optimised intmath.h

Use of the ARM optimised intmath.h was accidentally dropped in 9734b8b.

Signed-off-by: Mans Rullgard <mans@mansr.com>

ffv1: update to ffv1 version 3

Based on code from Carl Eugen Hoyos, Michael Niedermayer and Paul B Mahol.

ffv1: propagate errors

ffv1: split decoder and encoder

avutil: Add a copy of ff_sqrt_tab back into avutil to restore ABI compatibility

Earlier versions of for instance of libavcodec expect this symbol to be
present in libavutil. This commit can be reverted after the next major
bump.

New shared builds of avcodec will link to the internal copy of the
table within that library, so those builds won't rely on this table
being present in avutil any longer either.

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

avutil: make some tables visible again

These tables are used for instance by older versions of libavcodec and
need to remain visible until the next SONAME bump.

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

avutil: remove inline av_log2 from public API

This removes inline av_log2 and av_log2_16bit from the public API,
instead exporting them as regular functions.  In-tree code still
gets the inline and otherwise optimised variants.

Signed-off-by: Mans Rullgard <mans@mansr.com>

celp_math: rename ff_log2 to ff_log2_q15

This name is more descriptive as the function returns a fixed-point
value with 15 fraction bits.

Signed-off-by: Mans Rullgard <mans@mansr.com>

configure: Group math functions into a separate variable

This avoids some duplication when listing and testing for the functions.

avutil/mem: K&R formatting cosmetics

avutil/lzo: K&R formatting cosmetics

fate-vc1: add dependencies

Signed-off-by: Mans Rullgard <mans@mansr.com>

ARM: fix overreads in neon h264 chroma mc

The loops were reading ahead one line, which could end up outside the
buffer for reference blocks at the edge of the picture.  Removing
this readahead has no measurable performance impact.

Signed-off-by: Mans Rullgard <mans@mansr.com>

rtsp: Make sure the ret variable is initialized in ff_rtsp_fetch_packet

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

gitignore: ignore files created by msvc

exp files are created in every build and contain export information
of the libraries. Both pdb and ilk are created during debug builds,
and contain linking and debug information used by MSVC.

Signed-off-by: Luca Barbato <lu_zero@gentoo.org>

fate: Add proper dependencies for the tests in video.mak

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

configure: Disable Snow decoder and encoder by default

Snow has serious unfixed bugs and no real-world use.

lzo: Drop obsolete fast_memcpy reference

build: Drop OBJS declaration for non-existing PCM_DVD encoder

mpeg4videodec: Disable frame multithreading for GMC, its not implemented at all

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

fate: Add proper dependencies in qt.mak

The output of make fate-list is identical to before.

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

fate: Add proper dependencies in lossless-video.mak

The output of make fate-list is identical to before.

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

indeo3: do not try to output more lines than we can fit

Internally chroma planes have multiple of four height while allocated image
planes might be smaller if CODEC_FLAG_EMU_EDGE is set. Thus we should not
output more lines of chroma than frame can accept.

Also the decoder can be safely switched to direct rendering now.

bmv: get a new frame on every decode_frame(), so we can use direct rendering

configure: fix libcdio check

The compiler/linker flags passed to check_lib2 should not be quoted.

Signed-off-by: Mans Rullgard <mans@mansr.com>

rtsp: Allow setting the reordering buffer size via an AVOption

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

rtsp: Vertically align a constant definition

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

rtp: Update the check for distinguishing between RTP and RTCP

Also add enums for more RTCP packet types, according to the IANA
list of registered types.

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

aac: fix build with hardcoded tables

aac_tablegen.h includes aac.h for the POW_SF2_ZERO definition, but
this also pulls in a raft of other headers, some of which are not
safe to use in code built with the host compiler.

Moving POW_SF2_ZERO to aac_tablegen_decl.h, where the declaration
of the array it relates to already resides, fixes the problems.

Signed-off-by: Mans Rullgard <mans@mansr.com>

fate: dependencies for screen codec tests

riff: Move functions around to be covered by appropriate #ifdefs

This fixes compilation with --disable-muxers.

avformat: Fix references to the removed function av_write_header in comments

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

changelog: Mention the MSVC DLL support

Also retroactively add a changelog entry to the 9beta1 list
for general MSVC support, which was present there already.

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

fate: add dependencies for misc microsoft codecs

Signed-off-by: Mans Rullgard <mans@mansr.com>

fate-twinvq: add dependencies

Signed-off-by: Mans Rullgard <mans@mansr.com>

fate-mpc: add dependencies

Signed-off-by: Mans Rullgard <mans@mansr.com>

fate-indeo: add dependencies

Signed-off-by: Mans Rullgard <mans@mansr.com>

fate-als: add dependencies

Signed-off-by: Mans Rullgard <mans@mansr.com>

fate: dependencies for demux tests

Signed-off-by: Mans Rullgard <mans@mansr.com>

fate: add a dependency helper macro

Signed-off-by: Mans Rullgard <mans@mansr.com>

Add support for building shared libraries with MSVC

This requires the makedef perl script by Derek, from the
c89-to-c99 repo. That scripts produces a .def file, listing
the symbols to be exported, based on the gcc version scripts
and the built object files.

To properly load non-function symbols from DLL files, the
data symbol declarations need to have the attribute
__declspec(dllimport) when building the calling code. (On mingw,
the linker can fix this up automatically, which is why it has not
been an issue so far. If this attribute is omitted, linking
actually succeeds, but reads from the table will not produce the
desired results at runtime.)

MSVC seems to manage to link DLLs (and run properly) even if
this attribute is present while building the library itself
(which normally isn't recommended) - other object files in the
same library manage to link to the symbol (with a small warning
at link time, like "warning LNK4049: locally defined symbol
_avpriv_mpa_bitrate_tab imported" - it doesn't seem to be possible
to squelch this warning), and the definition of the tables
themselves produce a warning that can be squelched ("warning C4273:
'avpriv_mpa_bitrate_tab' : inconsistent dll linkage, see previous
definition of 'avpriv_mpa_bitrate_tab').

In this setup, mingw isn't able to link object files that refer to
data symbols with __declspec(dllimport) without those symbols
actually being linked via a DLL (linking avcodec.dll ends up with
errors like "undefined reference to `__imp__avpriv_mpa_freq_tab'").
The dllimport declspec isn't needed at all in mingw, so we simply
choose not to declare it for other compilers than MSVC that requires
it. (If ICL support later requires it, the condition can be extended
later to include both of them.)

This also implies that code that is built to link to a certain
library as a DLL can't link to the same library as a static library.
Therefore, we only allow building either static or shared but not
both at the same time. (That is, static libraries as such can be,
and actually are, built - this is used for linking the test tools to
internal symbols in the libraries - but e.g. libavformat built to
link to libavcodec as a DLL cannot link statically to libavcodec.)

Also, linking to DLLs is slightly different from linking to shared
libraries on other platforms. DLLs use a thing called import
libraries, which is basically a stub library allowing the linker
to know which symbols exist in the DLL and what name the DLL will
have at runtime.

In mingw/gcc, the import library is usually named libfoo.dll.a,
which goes next to a static library named libfoo.a. This allows
gcc to pick the dynamic one, if available, from the normal -lfoo
switches, just as it does for libfoo.a vs libfoo.so on Unix. On
MSVC however, you need to literally specify the name of the import
library instead of the static library.

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

avcodec: Rename avpriv_frame_rate_tab to ff_mpeg12_frame_rate_tab

This table doesn't need to be shared with libavformat any longer.
Add mpeg12 to the name to make it less ambiguous, while renaming it.

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

gxf: Add a local copy of the relevant parts of the frame rate table

This avoids having to share this table across the library
boundaries.

This shared table used to be problematic, if always declaring
all exported data symbols with the dllimport attribute (even
while building that same library), since it needs to be a
link-time constant when it is used in AVCodec declarations
(in mpeg12enc.c).

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

configure: Split out msvc as a separate target OS

The name mingw32 as target OS is both misleading, and very little
of the target OS specific settings actually match.

Since the target OS default is set based on uname, the default
(which on MSYS is set to mingw) is overridden by --toolchain=msvc.

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

aviobuf: Remove a senseless ifdef in avio_seek

This seemed to assume that one never used writing avio unless
muxers or networking was enabled.

This ifdef is a remnant since 8fa641f8.

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

mips64: mark hi/lo registers clobbered in MAC64/MLS64 macros

Signed-off-by: Mans Rullgard <mans@mansr.com>

fate: list lavfi tests in a makefile

Signed-off-by: Mans Rullgard <mans@mansr.com>

fate: dependencies for seek tests

Signed-off-by: Mans Rullgard <mans@mansr.com>

fate: handle lavf test dependencies entirely in make

This makes the lavf tests depend on all codecs and formats they use.

Signed-off-by: Mans Rullgard <mans@mansr.com>

fate: dependencies for acodec tests

Signed-off-by: Mans Rullgard <mans@mansr.com>

fate: dependencies for vsynth tests

This makes the vsynth tests run only if the required codecs and
formats are enabled.

Signed-off-by: Mans Rullgard <mans@mansr.com>

fate: add macros useful for conditionally enabling things

Signed-off-by: Mans Rullgard <mans@mansr.com>

libmp3lame: resize the output buffer if needed

The LAME API documentation for the required buffer size refers to the size for
a single encode call. However, we store multiple frames in the same output
buffer but only read 1 frame at a time out of it. As a result, the buffer size
given in lame_encode_buffer() is actually smaller than what it should be.
Since we do not know how many frames it will end up buffering, it is best to
just reallocate if needed.

configure: Add support for Tilera processors

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

wavdec: check size before reading the data, not after.

wav: do not fail on empty INFO tags

Fixes Bug 379

CC: libav-stable@libav.org

lavf: split wav muxer and demuxer into separate files.