Signed-off-by: NMichael Niedermayer <michaelni@gmx.at>

This was broken in 095be4fb - samples+ch (for the previous
non-planar case) equals &samples_p[ch][0]. The confusion
probably stemmed from the IMA WAV case where it originally
was &samples[avctx->channels + ch], which was correctly
changed into &samples_p[ch][1].

Fixes part of Ticket3701
Signed-off-by: NMichael Niedermayer <michaelni@gmx.at>

MSVC apparently doesnt support fmin() either

Suggested/Found-by: ubitux, Daemon404, nevcairiel
Signed-off-by: NMichael Niedermayer <michaelni@gmx.at>

Signed-off-by: NMichael Niedermayer <michaelni@gmx.at>

Found-by: ubitux
Signed-off-by: NMichael Niedermayer <michaelni@gmx.at>

Signed-off-by: NMichael Niedermayer <michaelni@gmx.at>

* cehoyos/master:
  Allow >2G for skip_initial_bytes.
Merged-by: NMichael Niedermayer <michaelni@gmx.at>

This is done by padding the coefficient buffer with 0s, because the order
may be only a multiple of 4, and the DSP function requires batches of 8.

However, no sample with such a case was found, so request one if it uses
that kind of order.

Approximate relative speedup depending on instruction set:
plain C: -6%
mmxext:  51%
sse2:    54%
Signed-off-by: NMichael Niedermayer <michaelni@gmx.at>

* commit '2f706519':
  libfdk-aac: Relicense the library wrappers to the ISC license
Merged-by: NMichael Niedermayer <michaelni@gmx.at>

* commit 'e7d6d0bf':
  mov: Export geotag metadata fields

Conflicts:
	libavformat/mov.c
Merged-by: NMichael Niedermayer <michaelni@gmx.at>

This reduces the number of different licenses used within libav,
and is preferrable since it has less ambiguous wordings than
the BSD license with respect to the duties of the user of the code.

Fraunhofer have now indicated that they're allowed to contribute
code under this license as well.
Signed-off-by: NMartin Storsjö <martin@martin.st>

The '?xyz' form is used by android devices (and according to apple
mailing list archives, also by older iOS devices). The 'loci' field
(defined in 3GPP 26.244) is used by recent iOS devices.

Even though the loci field can contain an altitude, it was plain
0 in my sample. Just export longitude and latitude, in a string
format matching the one used by the '?xyz' metadata field.
Signed-off-by: NMartin Storsjö <martin@martin.st>

This reduces differences with the fork
Signed-off-by: NJames Almer <jamrial@gmail.com>
Signed-off-by: NMichael Niedermayer <michaelni@gmx.at>

Fixes Ticket3698
Signed-off-by: NMichael Niedermayer <michaelni@gmx.at>

Fixes Ticket3700
Signed-off-by: NMichael Niedermayer <michaelni@gmx.at>

This makes the mp3 decoder produce the same result when repeatly flushing and decoding
Suggested-by: NDale Curtis <dalecurtis@chromium.org>
Signed-off-by: NMichael Niedermayer <michaelni@gmx.at>

APE is not the sole codec using scalarproduct_and_madd_int16.
Signed-off-by: NMichael Niedermayer <michaelni@gmx.at>

* commit 'd816e125':
  dict: const correctness for av_dict_get() and av_dict_copy()

Conflicts:
	libavutil/dict.c
	libavutil/dict.h

See: e12a7324Merged-by: NMichael Niedermayer <michaelni@gmx.at>

Fixes Ticket3658
Signed-off-by: NMichael Niedermayer <michaelni@gmx.at>

this filter is the same as showspectrum but with constant Q transform,
so frequency is spaced logarithmically

adpcm_ima_qt does not produce reproducible results, so it is temporarily
disabled (see #3701).
Signed-off-by: NTimothy Gu <timothygu99@gmail.com>
Signed-off-by: NMichael Niedermayer <michaelni@gmx.at>

Signed-off-by: NTimothy Gu <timothygu99@gmail.com>
Signed-off-by: NMichael Niedermayer <michaelni@gmx.at>

This prevents all results from being declared whenever the function is called.
Signed-off-by: NTimothy Gu <timothygu99@gmail.com>
Signed-off-by: NMichael Niedermayer <michaelni@gmx.at>

Untested due to lack of hardware
Signed-off-by: NMichael Niedermayer <michaelni@gmx.at>

Before: 154861 decicycles
After:  152912 decicycles
Signed-off-by: NMichael Niedermayer <michaelni@gmx.at>

An invalid entry already has the property of having a negative number
of bits, so remove the check on the reserved value, and rearrange the
code as a consequence.

346800 decicycles in 422, 262079 runs, 65 skips
168197 decicycles in gray, 262077 runs, 67 skips
Overall time: 7.878s

319076 decicycles in 422, 262096 runs, 48 skips
159875 decicycles in gray, 262057 runs, 87 skips
Overall time: 7.394s
Signed-off-by: NMichael Niedermayer <michaelni@gmx.at>

10496 -> 9105 cycles.
Signed-off-by: NMichael Niedermayer <michaelni@gmx.at>

It's no longer used inside another specific macro, so rename it.
Also remove duplicated definition and realign code.
Signed-off-by: NMichael Niedermayer <michaelni@gmx.at>

Found-by: NNiels Möller <nisse@lysator.liu.se>
Signed-off-by: NMichael Niedermayer <michaelni@gmx.at>

These where intended to maintain the previous behavior before dca_dmix_code()
but it is unclear (to me) which way is correct and no sample seem to trigger
the case, also they are incomplete for the purprose of error checking
Found-by: NNiels Möller <nisse@lysator.liu.se>
Signed-off-by: NMichael Niedermayer <michaelni@gmx.at>

* cehoyos/master:
  Fix rc_max_rate documentation.
  Fix some fate filter tests with --target-path.
Merged-by: NMichael Niedermayer <michaelni@gmx.at>

The mpegvideo decoder sets rc_max_rate since 25b7aa98.

* commit '39ec5e1c':
  avconv: Report the codec and the encoder separately
Merged-by: NMichael Niedermayer <michaelni@gmx.at>

Avoid buffer overruns when processing some MOV files with the amerge
filter. Files produced by Adobe Premiere Pro CC have up to one second of audio
not interleaved. With common settings (<= 48kHz) that makes up to 47 frames so
a queue length of 64 makes sense.

Fixes ticket #3510.
Reviewed-by: NNicolas George <george@nsup.org>
Signed-off-by: NMichael Niedermayer <michaelni@gmx.at>

Signed-off-by: NMichael Niedermayer <michaelni@gmx.at>

Before:
179436 decicycles in rgb, 32735 runs, 33 skips
Stripped object size: 23188

After:
154861 decicycles in rgb, 32738 runs, 30 skips
Stripped object size: 22736
Signed-off-by: NMichael Niedermayer <michaelni@gmx.at>

When the joint table does not contain a valid entry, the decoding restarts
from scratch. By implementing the trick of jumping to the 2nd level of the
individual table (and inlining the whole), a speed improvement of 5-10%
is possible.

On a 1000-frames YUV4:2:0 video, before:
362851 decicycles in 422, 262094 runs, 50 skips
182488 decicycles in gray, 262087 runs, 57 skips
Object size: 23584
Overall time: 8.377

After:
346800 decicycles in 422, 262079 runs, 65 skips
168197 decicycles in gray, 262077 runs, 67 skips
Object size: 23188
Overall time: 7.878
Signed-off-by: NMichael Niedermayer <michaelni@gmx.at>