dict: add av_dict_count()

This adds a function to retrieve the number of entries in a
dictionary and updates the places directly accessing what should
be an opaque struct to use this new function instead.

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

g723.1: fix addition overflow

This addition must be done as 64-bit to avoid overflow and for
the subsequent clipping to be meaningful.

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

g723.1: simplify and fix multiplication overflow

In 16-bit arithmetic, x * 0xffffc is simply x * -4 with extra overflows,
(and the constant was probably meant to be 0xfffc).  Combined with the
shift, this simplifies to -x >> 1.  Finally, clearing the low two bits
with a 32-bit mask and switching to a 32-bit type allows more efficient
code on 32-bit machines.

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

g723.1: deobfuscate an expression

(x << 2) - x is just an optimisation of 3 * x the compiler is
perfectly capable of doing on its own.

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

g723.1: remove unused #includes

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

ARM: add missing "cc" clobber in av_clipl_int32_arm()

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

rtmp: Factorize the code by adding handle_invoke_error

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

rtmp: Factorize the code by adding handle_invoke_status

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

rtmp: Factorize the code by adding handle_invoke_result

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

libavutil: remove unused av_abort() macro

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

ffmenc: replace if/abort with assert()

The condition is trivially true, but keeping the assert() is
sensible to avoid FFM_HEADER_SIZE ever getting out of sync with
the actual code.

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

libavutil: drop offsetof() fallback definition

The only compiler I have that does not define the standard
offsetof() macro is "Bruce's C Compiler", a simple compiler
for producing 8/16-bit 8086 code, usually for use in early
stages of PC booting.

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

libavutil: drop fallback definitions of INTxx_MIN/MAX

This list is incomplete (we also use UINT16_MAX), so there does
not appear to be any system we care about that needs these.

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

configure: Check for a sctp struct instead of just the header

This fixes build failures on debian/kfreebsd, which has the
sctp.h header, but it is currently broken (a cpp test succeeds,
but a compile test fails), see http://bugs.debian.org/684330 for
details.

Also remove the checked item from HAVE_LIST, since the corresponding
HAVE_* define isn't used by the source code.

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

configure: suncc: Add -xc99 to dependency flags, required on Solaris

doxygen: Fix function parameter names to match the code

doc: Drop obsolete shared libs cflags hint to workaround Cygwin gcc bugs

swf: Move shared table out of the header file

swf: Move swf_audio_codec_tags table to the only place it is used

fate: add G.723.1 decoder tests

motion_est: drop inline from sad_hpel_motion_search()

This function is only ever called through a function pointer,
so marking it inline makes no sense.

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

motion_est: remove unused macros

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

motion_est: remove useless no_motion_search() function

At both places this function is called, mb_[xy] == s->mb_[xy]
making the call together with following code equivalent to
simply assigning zeros.

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

lagarith: frame multithreading

About 2x speedup going from 1 to 2 threads.
1.7s to 0.85s on foreman CIF.

Signed-off-by: Janne Grunau <janne-libav@jannau.net>

doxygen: qdm2: Drop documentation for non-existing function parameters

build: add HOSTOBJS to SUBDIR_VARS list

Even though HOSTOBJS are not referenced directly in subdirectory Makefile
snippets right now, robustness requires resetting the variable contents.

mpegvideo: reduce excessive inlining of mpeg_motion()

The main benefit of inlining this function is from constant
propagation for the 'field_based' argument.  Instead of inlining
all calls, create two versions of the function for field_based
values of 0 and 1.

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

mpegvideo: convert mpegvideo_common.h to a .c file

This file defines a single, huge function, MPV_motion(), which
although being declared inline is not actually inlined by the
compiler (for good reason).  There is thus no sense in defining
this function in a header file, resulting in multiple copies of
it in the final library.

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

build: factor out mpegvideo.o dependencies to CONFIG_MPEGVIDEO

This adds a hidden config variable for the mpegvideo.o dependency
and selects from the codecs which require it.

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

Move MASK_ABS macro to libavcodec/mathops.h

This macro is only used in two places, both in libavcodec, so this
is a more sensible place for it.

Two small tweaks to the macro are made:

- removing the trailing semicolon
- dropping unnecessary 'volatile' from the x86 asm

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

x86: move MANGLE() and related macros to libavutil/x86/asm.h

These x86-specific macros do not belong in generic code.

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

x86: rename libavutil/x86_cpu.h to libavutil/x86/asm.h

This puts x86-specific things in the x86/ subdirectory where they
belong.

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

aacdec: Don't fall back to the old output configuration when no old configuration is present.

Fixes MP4 files where the first frame is broken.

rtmp: Add message tracking

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

rtsp: Support mpegts in raw udp packets

This is basically the same way as mpegts packets are parsed in
rtpdec.c.

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

rtsp: Support receiving plain data over UDP without any RTP encapsulation

EvoStream Media Server can serve data in this format, and
VLC/live555 already supports it.

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

rtpdec: Remove an unused include

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

rtpenc: Remove an av_abort() that depends on user-supplied data

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

vsrc_movie: discourage its use with avconv.

avconv: allow no input files.

It is now possible to use lavfi sources.

avconv: prevent invalid reads in transcode_init()

avconv: rename OutputStream.is_past_recording_time to finished.

The new name is shorter and more accurate, since this variable is no
longer used only for checking recording time constraint.

avconv: split the code for processing input packets out of transcode()

avconv: send EOF to lavfi even if flushing the decoder fails

avconv: get rid of pointless temporary variable.

avconv: simplify transcode().

Operate with a pointer to InputFile instead of its index in input_files.

avconv: cosmetics

Replace for (;foo == 0;) with while (!foo)
This is prettier.

avconv: replace no_packet array in transcode() with a var in InputStream

This simplifies splitting code for reading from input out of
transcode().

avconv: remove unused variable from InputFile.

avconv: remove commented out cruft.

avconv: maintain sync on lavfi outputs.

Before this commit, poll_filters() reads all frames available on each
lavfi output. This does not work for lavfi sources that produce
an unlimited number of frames, e.g. color and similar.

With this commit, poll_filters() reads from output with the lowest
timestamp and returns to wait for more input if no frames are available
on it.

build: cosmetics: Reorder some lists in a more logical fashion

x86: pngdsp: Fix assembly for OS/2

The a.out object format does not allow aligning sections.
On OS/2 LD aligns sections to 16 bytes.

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

fate: add test for RTjpeg in nuv with frameheader

Renames the old test to allow using fate-nuv as group for all
nuv tests.

rtmp: send check_bw as notification

Set the transactionId to 0 to signal the server a return _result/_error
is not expected.

Solves the problems experienced with strict rtmp servers.

g723_1: clip argument for 15-bit version of normalize_bits()

It expects maximum value to be 32767 but calculations in scale_vector()
which uses this function can give it ABS(-32768) which leads to wrong
result and thus clipping is needed.

g723_1: use all LPC vectors in formant postfilter

Due to some mistake LPC vector for the first subframe was used for all
subframes instead of their own LPC vectors.

id3v2: Support v2.2 PIC

id3 v2.2 uses image format ("JPG","PNG") instead of mimetypes.

Currently, the attached picture is skipped because the format string
does not match a known picture mimetype.

This patch fixes this behaviour.

Signed-off-by: Mohammad Alsaleh <msal@tormail.org>
Signed-off-by: Anton Khirnov <anton@khirnov.net>

avplay: fix build with lavfi disabled.

Issue introduced in 67339f6e.

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

avconv: split configuring filter configuration to a separate file.

avconv: split option parsing into a separate file.

mpc8: do not leave padding after last frame in buffer for the next decode call

mpegaudioenc: list supported channel layouts.

mpegaudiodec: don't print an error on > 1 frame in a packet.

It's a perfectly normal situation, nothing to spam about.

api-example: update to new audio encoding API.

configure: add --enable/disable-random option

This allows creating random configurations which is useful for
testing purposes.

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

doc: cygwin: Update list of FATE package requirements

build: Remove all installed headers and header directories on uninstall

build: change checkheaders to use regular build rules

Many compilers need special flags to compile *.h files as regular
source code, if they will do so at all.  Rather than hoping all
compilers will have such a flag and adding mappings for it, create
wrapper .c files for test building single headers.

This allows using the regular rule for compiling C files without the
need for special flags, and it also provides proper dependency tracking
for these objects.

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

rtmp: Add a new option 'rtmp_subscribe'

This option specifies the name of live stream to subscribe.
Defaults to rtmp_playpath.

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

rtmp: Add support for subscribing live streams

When streaming live streams using the Akamai, Edgecast or Limelight CDN,
players cannot simply connect to the live stream. Instead, they have to
subscribe to it, by sending an FC Subscribe call to the server.

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

rtmp: Factorize simultaneous URLProtocol / AVClass declarations with a macro

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

ARM: use Q/R inline asm operand modifiers only if supported

Some compilers do not support the Q/R modifiers used to access
the low/high parts of a 64-bit register pair.  Check for this
and disable all uses of it when not supported.

Fixes bug #337.

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

configure: Add support for e500mc CPU

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

x86: fix build with nasm 2.08

It appears that something goes wrong in old nasm versions when the
%+ operator is used in the last argument of a macro invocation and
this argument is tested with %ifdef within the macro.  This patch
rearranges the macro arguments such that the %+ operator is never
used in the last argument.

x86: use nop cpu directives only if supported

nasm does not support 'CPU foonop' directives.  This adds a configure
test for the directive and uses it only if supported.

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

x86: fix rNmp macros with nasm

For some reason, nasm requires this.  No harm done to yasm.

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

build: add trailing / to yasm/nasm -I flags

nasm requires a trailing / on paths specified with -I.
It does no harm with yasm.

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

x86: use 32-bit source registers with movd instruction

yasm tolerates mismatch between movd/movq and source register size,
adjusting the instruction according to the register.  nasm is more
strict.

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

x86: add colons after labels

nasm prints a warning if the colon is missing.

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

Replace all CODEC_ID_* with AV_CODEC_ID_*

lavc: add AV prefix to codec ids.

build: fix library installation on cygwin

This installs libraries using the proper names and locations,
generates an import lib for the DLL, and drops no longer needed
linker flags.

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

mpc8: add a flush function

Ensures that the next frame decoded after seeking will be decoded as a
keyframe.

mpc8: set packet duration and stream start time instead of tracking frames

Fixes mpc8 timestamps.

nuv: K&R formatting cosmetics

build: generalise rules and variable settings for av* programs

This simplifies adding extra flags for individual programs
and also allows more than one object file per program.

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

nuv: check RTjpeg header for validity

CC: libav-stable@libav.org

Revert "nuv: check per-frame header for validity."

The check is bogus since the nuv frameheader is already skipped
and the (decompressed) RTjpeg header is checked.

This reverts commit f6afacdb3b708720c9fb85984b4f7fdbca2b2036.

CC: libav-stable@libav.org

imc: remove unused field IMCContext.one_div_log2

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

imc: fix size of a memset()

IMCContext was changed from an array to a pointer in 66b84e4,
but this memset() was not updated.

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

imc: remove empty if() block

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

fate: simplify variable setting filter.mak

This removes some needless indirection and duplication.

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

lavf: Declare an AVRational struct without a struct literal

At this place, the normal way of initializing a struct works
fine, there's no need for a struct literal.

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

x86: h264_idct: Rename x264_add8x4_idct_sse2 --> h264_add8x4_idct_sse2

rational: add av_inv_q() returning the inverse of an AVRational

This allows simplifying a few expressions.

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

dpx: Make start offset unsigned

Some corrupted files would end up with a negative offset,
and segfault.

Fixes bug #177.

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

lavfi: properly signal out-of-memory error in ff_filter_samples

Found with a clang-scan report on http://fate.libav.org/csa/

cosmetics: Fix a few switched periods and linebreaks

Based on a patch by Piotr Bandurski.

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

zerocodec: Fix memleak in decode_frame

If there was a failure inflating, or reinitializing
the zstream, the current frame's buffer would be lost.

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