GitLab

Replace
     a = _mm_alignr_epi8(b, a, n);
with
     a = _mm_alignr_epi8(a, b, n);

The resulting code is very slightly faster and the binaries slightly
smaller.

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

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

Previously this function would abort or crash if passed `NULL` for
the encoder. Now it just returns without crashing or aborting which
is far more sane behaviour (ie much like FLAC__stream_encoder_delete).

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/28

* Furthermore, use NDEBUG globally to detect the presence
  of building with more debug output information.
  AX_CHECK_ENABLE_DEBUG is easier to use, and nowadays
  Gnome has also switched to it from its own custom solution.
Signed-off-by: Erik de Castro Lopo <erikd@mega-nerd.com>

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

This version should be logically identical to the previous version
but prevents a false-positive from the cppcheck static analysis
tool.

This adds a new msvc2005_int.h header file which is "force included"
as needed by the MSVS project file.

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

* Remove <stdint.h> because MSVC 2005 (and probably 2008) don't have it.
* Fix pointer difference caclulation between `nameend` and `nextchar`.

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

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

Found using the Makefile.lite build system.

The orignal code had `unsigned long` inside `#ifdef MSVC`. It was then changed
to `uint64_t` which was incorrect because on Windows `sizze long == 4`.
Change it now to `uint32_t` which is always correct regardless of OS, compiler
or architecture.

cpu.c was the only source to use memory.h instead of string.h.
Signed-off-by: Erik de Castro Lopo <erikd@mega-nerd.com>

* HTML files should be installed to $(htmldir), and $(docdir) should
  not be changed, as this is a user flag in the GNU conventions.
Signed-off-by: Erik de Castro Lopo <erikd@mega-nerd.com>

* This is required, as otherwise -Wl,--as-needed could fail.
Signed-off-by: Erik de Castro Lopo <erikd@mega-nerd.com>

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

These fixes are in the public API, but FLAC would have been extremely
unlikely to work in any system where `unsigned` was not 32 bits.
Therefore we do not treat this as an API change.

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

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

As pointed out by Ozkan Sezer, on some platforms `int32_t` is actually
a typedef for `long` so `unsigned` cannot be used interchangably with
`FLAC__uint32`. Fix is to switch from `unsigned` to explicit sized ISO
C types defined in <stdint.h>.

Not all linux hosts match the *-pc-linux-gnu wildcard, causing build
failures for older glibc versions where we need to link with -lrt for
clock_gettime - E.G.:

 - arm-unknown-linux-musleabihf
 - powerpc-unknown-linux-gnuspe
 - bfin-linux-linux-uclibc
 ..
Signed-off-by: Peter Korsgaard <peter@korsgaard.com>
Signed-off-by: Erik de Castro Lopo <erikd@mega-nerd.com>

Patch-from: Christian Weisgerber <naddy@mips.inka.de>

As pointed out by github user @randomascii, the `compression_levels_`
struct could and should be const.

Closes: https://github.com/xiph/flac/issues/26

Patch-from: "lvqcl.mail" <lvqcl.mail@gmail.com>
Closes: https://sourceforge.net/p/flac/bugs/447/

Patch-from: Janne Hyvärinen <cse@sci.fi>

Pointed out by c.helmrich@ecodis.de. It was added during debugging
in commit c06a4496, but was never meant to stay.

* Generate `test/common.sh` from `test/common.sh.in`.
* Move `is_win` setting to `test_common.sh`.

Patch-from: Robert Kausch <robert.kausch@freac.org>

Patch-from: Robert Kausch <robert.kausch@freac.org>

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