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.

wav muxer: write metadata

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

riff: do not write empty INFO tags

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

lavf: move RIFF INFO tag writing from avienc to riff

It will be useful in the wav muxer.

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

avconv: fix disabling auto mappings with -map_metadata

CC: libav-stable@libav.org

avcodec_encode_audio(): fix invalid free

Since 2bc0de385, AVFrame needs to be initialized
before calling avcodec_get_frame_defaults().

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

pcm-mpeg: correct bitrate calculation

Bitrate calculation is off since the bluray spec always specifies
an even number of coded channels. This was honored in the decoder,
but not for bitrate calculation.

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

ffv1: K&R formatting cosmetics

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

fate: Add rangecoder test

network: #include stdint.h in network.h

This header uses various types provided by stdint.h without
explicitly including it.

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

nut: export codec_tag provided by rawvideo

Raw audio does not provide valid audio tags while rawvideo does.
The fate refs have to be updated because it undoes the previous tag
change.

avserver: move avserver-specific code from ffmdec.c to avserver.c

This way avserver only depends on the data structures of the ffm
demuxer, which it already does, and not also on private functions
being exported by the library.

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

build: simplify linking tools with cmdutils.o

This avoids repeating cmdutils.o in both the prerequisites and
the link command.

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

tiny_psnr: fix range calculation for sample size of 32 bits

For a sample size of 32 bits, the shift would overflow producing
undefined results.  Incidentally, in the only test currently using
32-bit samples, the output matches the reference exactly on most
systems meaning the bad 'max' value is never used.

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

tiny_psnr: check for specified sample size less than 1

A zero or negative sample size is impossible and should be
reported as an error.

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

fate: improve md5sum utility selection

The 'md5sum' command is used with the -b flag so the presence test
must also use this flag.

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

rangecoder-test: Drop timer output that clutters stderr

rangecoder-test: Return in case of an error

build: simplify enabling of compat objects

Add a configure function to pull in a compat object and set up
redirects in one operation.  This avoids duplicating conditions
across configure and makefiles.

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

configure: simplify argument handling in check_ld

Use the existing filter functions instead of open-coding the
separation of general flags and libraries.

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

configure: simplify get_version() function

awk alone can do this, no need for grep.

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

build: support asan and tsan toolchain shortcuts

rmdec: Move SIPR code shared with Matroska demuxer to a separate file

swscale: try to use mmap only if available

Some systems, e.g. Minix, have sys/mman.h defining MAP_ANONYMOUS without
providing (working) mmap and friends.  The mmx filter generation code
checks only for MAP_ANONYMOUS, not for availability of mmap itself which
leads to build errors on aforementioned systems.

This changes the conditional compilation to use mmap only if all the
required functions are available.

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

configure: check for mprotect

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

wmapro: use planar sample format

wmalossless: output in planar sample format

wmadec: use float planar sample format output

shorten: use planar sample format

lavc: update documentation for AVFrame.extended_data

avutil: Do not make ff_ symbols globally visible.

avutil: Rename ff_set_systematic_pal2() ---> avpriv_set_systematic_pal2()

Functions used across libraries should have avpriv_ and not ff_ prefixes.

build: tms470: work around glibc math.h problems

The glibc definitions of INFINITY and NAN do not work with the
tms470 compiler, nor do our usual fallbacks.

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

configure: improve tms470 compiler usage with glibc

Apply flags to work around glibc quirks only if glibc is detected,
and add a few more such flags.

Do not mess with as/ld settings in probe_cc.  This is not the
proper place.

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

configure: tms470: add mapping for -mfpu=vfpv3-d16 flag

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

configure: recognise Minix as OS

No special setup is required for Minix.

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

configure: work around bug in ash shell

The ash 'test' builtin misbehaves if the first operand of a binary
operator looks like a unary operator.

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

eval-test: make table static const

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

lavr: handle clipping in the float to s32 conversion

We cannot clip to INT_MAX because that value cannot be exactly
represented by a float value and ends up overflowing during conversion
anyway. We need to use a slightly smaller float value, which ends up
with slightly inaccurate results for samples which clip or nearly clip,
but it is close enough. Using doubles as intermediates in the conversion
would be more accurate, but it takes about twice as much time.

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

nut: support pcm codecs not mapped in avi

The native tags will be used when available.

nut: prioritize native tags

Use native tags instead of avi ones, simplifies a lot raw video codecs
handling.

vc1: Use codec ID from AVCodecContext while parsing frame header

This fixes a segfault with samples that I have (both of them MPEG-TS). Looks like
avctx->codec is not being set during parsing.

Signed-off-by: Kostya Shishkov <kostya.shishkov@gmail.com>

avplay: support mid-stream sample rate changes

Resample to the rate that was configured in SDL.

avplay: use audio parameters from the decoded frame instead of AVCodecContext

dca: allocate a secondary buffer for extra channels when downmixing

The output AVFrame buffer only has data for the downmix channels.
Fixes a segfault when decoding dca with request_channels == 2.

configure: use utilities from /usr/xpg4/bin if it exists

Solaris defaults to non-standard utilities (grep, sed, ...) with
proper ones being in /usr/xpg4/bin.  Prefixing PATH with this
directory when it exists ensures we get correct variants.

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

avstring-test: fix memory leaks

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

rtpdec_xiph: fix function return type

parse_packed_headers() returns either zero or a negative error code
so its return type must be signed.

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

smjpeg: fix type of 'ret' variable in smjpeg_read_packet()

The 'ret' variable is used for negative error codes so it should
be a signed type.

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