Found-by: jamrial
Signed-off-by: NMichael Niedermayer <michael@niedermayer.cc>

Signed-off-by: NPaul B Mahol <onemda@gmail.com>

Signed-off-by: NPaul B Mahol <onemda@gmail.com>

Signed-off-by: NPaul B Mahol <onemda@gmail.com>

Signed-off-by: NPaul B Mahol <onemda@gmail.com>

Signed-off-by: NPaul B Mahol <onemda@gmail.com>

It makes no sense whatsoever to do this at each function call; we
already have a table for this.

Yields a 2x improvement in find_min_book (x86-64, Haswell+GCC):
ffmpeg -i sin.flac -acodec aac -y sin.aac
find_min_book
old
    605 decicycles in find_min_book, 8388453 runs,    155 skips.9x
    606 decicycles in find_min_book,16776912 runs,    304 skips.9x
    607 decicycles in find_min_book,33553819 runs,    613 skips.2x
    607 decicycles in find_min_book,67107668 runs,   1196 skips.3x
    607 decicycles in find_min_book,134215360 runs,   2368 skips3x

new
    359 decicycles in find_min_book, 8388552 runs,     56 skips.3x
    360 decicycles in find_min_book,16777112 runs,    104 skips.1x
    361 decicycles in find_min_book,33554218 runs,    214 skips.4x
    361 decicycles in find_min_book,67108381 runs,    483 skips.5x
    361 decicycles in find_min_book,134216725 runs,   1003 skips5x

and more importantly a non-negligible speedup (~ 8%) to overall AAC encoding:
old:
ffmpeg -i sin.flac -acodec aac -strict -2 -y sin_new.aac  6.82s user 0.03s system 104% cpu 6.565 total
new:
ffmpeg -i sin.flac -acodec aac -strict -2 -y sin_old.aac  6.24s user 0.03s system 104% cpu 5.993 total

This also improves accuracy of the expression by ~ 2 ulp in some cases.
Reviewed-by: NDerek Buitenhuis <derek.buitenhuis@gmail.com>
Reviewed-by: NRostislav Pehlivanov <atomnuker@gmail.com>
Signed-off-by: NGanesh Ajjanagadde <gajjanag@gmail.com>

Signed-off-by: NMichael Niedermayer <michael@niedermayer.cc>

Signed-off-by: NMichael Niedermayer <michael@niedermayer.cc>

Fixes ticket #5271.

Signed-off-by: NMichael Niedermayer <michael@niedermayer.cc>

This fixes a data race warning by ThreadSanitizer.
FrameThreadContext.die is read by all the worker threads but is not
protected by any mutex. Move it to PerThreadContext so that each worker
thread reads its own copy of |die|, which can then be protected with
PerThreadContext.mutex.
Signed-off-by: NWan-Teh Chang <wtc@google.com>
Signed-off-by: NRonald S. Bultje <rsbultje@gmail.com>

This was a regression introduced by commit e7345abe which
enabled full use of the allocated packet but due to the overhead of
using field coding the buffer was too small and triggered warnings and
crashes.
Signed-off-by: NRostislav Pehlivanov <atomnuker@gmail.com>

The fact that now all quantization indices costs are cached justifies
storing 20 more integers in a structure already allocated on heap.
Signed-off-by: NRostislav Pehlivanov <atomnuker@gmail.com>

Signed-off-by: NRostislav Pehlivanov <atomnuker@gmail.com>

Signed-off-by: NMichael Niedermayer <michael@niedermayer.cc>

Add some more verbose info regarding why the imprecise and slow floor(x+0.5) hack
is used; helpful for future maintenance.
Reviewed-by: NMichael Niedermayer <michael@niedermayer.cc>
Signed-off-by: NGanesh Ajjanagadde <gajjanag@gmail.com>

Signed-off-by: NMichael Niedermayer <michael@niedermayer.cc>

Signed-off-by: NMichael Niedermayer <michael@niedermayer.cc>

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

Signed-off-by: NJames Almer <jamrial@gmail.com>

* commit 'cae448cf':
  aviobuf: add a private data struct for avio_open()ed contexts
Merged-by: NDerek Buitenhuis <derek.buitenhuis@gmail.com>

This fixes -read_ahead_limit
Signed-off-by: NMichael Niedermayer <michael@niedermayer.cc>

Fixes probing failure
Signed-off-by: NMichael Niedermayer <michael@niedermayer.cc>

* commit '832a202c':
  protocols: make the list of protocols static
Merged-by: NDerek Buitenhuis <derek.buitenhuis@gmail.com>

It is current disabled.
Signed-off-by: NDerek Buitenhuis <derek.buitenhuis@gmail.com>

* commit '7d61dc95':
  lavf: move urlcontext_child_class_next() to protocols.c
Merged-by: NDerek Buitenhuis <derek.buitenhuis@gmail.com>

* commit '0fa00d05':
  lavf: move avio_enum_protocols() to protocols.c
Merged-by: NDerek Buitenhuis <derek.buitenhuis@gmail.com>

This commit also disables the async fate test, because it
used internal APIs in a non-kosher way, which no longer
exists.

* commit '2758cded':
  lavf: reorganize URLProtocols
Merged-by: NDerek Buitenhuis <derek.buitenhuis@gmail.com>

* commit '225e84e7':
  hls: disallow opening nested files in child demuxers
Merged-by: NDerek Buitenhuis <derek.buitenhuis@gmail.com>

* commit 'e192cd9c':
  smoothstreamingenc: do not open the files as read+write
Merged-by: NDerek Buitenhuis <derek.buitenhuis@gmail.com>

* commit 'd082078a':
  dashenc: eliminate ffurl_* usage
Merged-by: NDerek Buitenhuis <derek.buitenhuis@gmail.com>

* commit '7fbb3b5b':
  lavf: use the io_open callbacks for files opened from open_input() as well
Merged-by: NDerek Buitenhuis <derek.buitenhuis@gmail.com>

This commit is no-op. The affected piece of code does not
exist in FFmpeg.

* commit '5efd9128':
  avprobe: do not call avio_close() on a custom context
Merged-by: NDerek Buitenhuis <derek.buitenhuis@gmail.com>

FATE tests have been updated to patch. They do not differ in
any meaningful way.

* commit 'dc6527ed':
  nutenc: do not use AVCodecContext.frame_size
Merged-by: NDerek Buitenhuis <derek.buitenhuis@gmail.com>