It is used for multiple codecs.

An alpha specifier outside the valid range results in a conversion from
double to long with undefined result.  Range-checking the double and
only converting it after it passes avoids this.

Fixes fate-parseutils errors on some systems.
Signed-off-by: Mans Rullgard <mans@mansr.com>

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

This helps tracking down the cause if this happens to be an issue.
Signed-off-by: Martin Storsjö <martin@martin.st>

We don't need to return an error if the directory already existed.
We make sure that the directory actually is properly writeable later
within ism_write_header by writing a manifest anyway.
Signed-off-by: Martin Storsjö <martin@martin.st>

This allows failing cleaner and earlier if unable to write to
the output directory.
Signed-off-by: Martin Storsjö <martin@martin.st>

This allows calling write_manifest from ism_write_header without
a forward declaration.
Signed-off-by: Martin Storsjö <martin@martin.st>

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

This code can be called with a NULL UrlContext if writing of the
trailer involves seeking.
Signed-off-by: Martin Storsjö <martin@martin.st>

Fixes Bug 373.

CC:libav-stable@libav.org

Anonymous structs cannot be forward declared and have no benefit.

This guarantees stable output for comparing test results.

Fixes Bug 203.

CC:libav-stable@libav.org

With all the VLAs gone, make sure they never come back.
Signed-off-by: Mans Rullgard <mans@mansr.com>

This gets rid of the variable-length scratch buffer by filtering 16
pixels at a time and writing directly to the destination.  The extra
loads this requires to load the source values are compensated by not
doing a round-trip to memory before shifting.
Signed-off-by: Mans Rullgard <mans@mansr.com>

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

Instead of using an evil VLA, fall back to C version when edge
emulation is needed.  MPEG4 GMC is a rarely used fringe feature
so the speed loss is an acceptable cost for safer code.
Signed-off-by: Mans Rullgard <mans@mansr.com>

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

Discontinuous transmission is an addition to VAD/VBR operation, that
allows to greatly reduce bitrate for silent chunks or stationary
noises.
Signed-off-by: Martin Storsjö <martin@martin.st>

This is more consistent with what the rest of Libav does.

This breaks API.

It has been superseded by lavr.

The variable is copied to subsequent threads at the same time, so this
may cause wrong ref_count[] values to be copied to subsequent threads.

This bug was found using TSAN.
Signed-off-by: Luca Barbato <lu_zero@gentoo.org>

ff_get_cpu_flags_x86() requires cpuid(), which is conditionally defined
elsewhere in the file.  Surrounding the function body with ifdefs allows
building even when cpuid is not defined.  An empty cpuflags mask is
returned in this case.

Now that there is CPU detection in YASM, there will always be one of
inline or external assembly enabled, which obviates the need to fall
back on CPU detection through compiler intrinsics.

This allows detecting CPU features with builds that have neither
gcc inline assembly nor the right compiler intrinsics enabled.

This works with gcc.  Other compilers might need to have a flag
mapping added.
Signed-off-by: Mans Rullgard <mans@mansr.com>