GitLab

Patch-from: lvqcl <lvqcl.mail@gmail.com>

This is an updtate to the fix for an MSVC 2015 bug in 94a61241.

Patch-from: lvqcl <lvqcl.mail@gmail.com>

The `inline` keyword is only available in C++ so include `share/compat.h`.
Signed-off-by: Erik de Castro Lopo <erikd@mega-nerd.com>

This is macro identifier is not defined, define it to `0`.

Signed-off-by: Erik de Castro Lopo <erikd@mega-nerd.com>

Signed-off-by: Erik de Castro Lopo <erikd@mega-nerd.com>

Cppcheck was incorrectly warning that `doubletmp` was being used
un-initialized. This was due to the fact that it was being used
and initialized in a pre-processor macro. Chaning the macro to
an inline function stops cppcheck from raising a warning.

This reverts commit b52fa0ec.

See: https://github.com/xiph/flac/commit/b52fa0ec6dc83ec0f430ff7bd7c80485c45728b0

Signed-off-by: Erik de Castro Lopo <erikd@mega-nerd.com>

The window amplitude is now  normalised to 1.0.
Signed-off-by: Erik de Castro Lopo <erikd@mega-nerd.com>

GCC 7 warns about `case` statements which are not separated by a
`break` statement. This warning can be fixed by adding a comment.

Remove redundant assignment before broadcasting into xmm registers.
Signed-off-by: Erik de Castro Lopo <erikd@mega-nerd.com>

Signed-off-by: Erik de Castro Lopo <erikd@mega-nerd.com>

This warning flag is gcc-specfic.
Signed-off-by: Erik de Castro Lopo <erikd@mega-nerd.com>

When decoding to WAV, the legacy wFormatTag of WAVE_FORMAT_PCM should
only be used if the bitwidth is 8 or 16. For all other bitwidths,
use WAVEFORMATEXTENSIBLE.

Two of the test programs/scripts also needed updating.

There were a number of specialized versions, but the default case
defined `MAX` and `MIN` instead of `flac_max` and `flac_min`.

Patch-from: "Michael W. Bombardieri" <mb@ii.net>

Leak reported by Secunia Research.

Patch-from: lvqcl <lvqcl.mail@gmail.com>

Signed-off-by: Erik de Castro Lopo <erikd@mega-nerd.com>
Closes: https://github.com/xiph/flac/pull/31

Signed-off-by: Erik de Castro Lopo <erikd@mega-nerd.com>

Switch from `asm` to `__asm__` which is better supported.

Patch-from: Olivier Tristan <o.tristan@uvi.net>

After the removal of the OS SSE detection stuff ia32_cpu_info()
and x86_64_cpu_info() became very similar. Merging them makes
sense.

Patch-from: lvqcl <lvqcl.mail@gmail.com>

Was failing to build without NASM.

Patch-from: Olivier Tristan <o.tristan@uvi.net>

Patch-from: lvqcl <lvqcl.mail@gmail.com>

Removes FLAC__lpc_restore_signal_16_intrin_sse2() which was faster
than than C code, but not faster than MMX-accelerated ASM functions.
It's also slower than the new SSE4.1 functions that were added by
the previous patch.

Patch-from: lvqcl <lvqcl.mail@gmail.com>

This patch removes FLAC__lpc_restore_signal_16_intrin_sse2().

It's faster than C code, but not faster than MMX-accelerated
ASM functions. It's also slower than the new SSE4.1 functions
that were added by the previous patch.
So this function wasn't very useful before, and now it's
even less useful. I don't see a reason to keep it.

Patch-from: lvqcl <lvqcl.mail@gmail.com>

Accelerates decoding of non-Subset 24-bit FLAC files (where lpc_order
> 12).

The improved function is FLAC__lpc_restore_signal_wide_intrin_sse41().
It requires SSE4.1 and it's used only by 32-bit libFLAC.

Patch-from: lvqcl <lvqcl.mail@gmail.com>