Fix the 'distclean' target.

Generated documentation needs to be removed by this target
for 'distcheck' to succeed.

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

Hoist a repeated conditional in the channel mapping code.

This is equivalent and just makes the code shorter.

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

autogen.sh : Make sure config.rpath exists.

doc: fix out-of-tree build

When building outside of the source tree, the Doxyfile needs to be
generated in the build tree and should point to the proper paths for
include directories and html footer.

The generated api files to install should also be taken from the build
tree instead of the source tree.

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

Revert parts of 96d081c1 that touched source code or autotools files.

Fix VC project files to use /objs/ directory instead of /obj/ directory.

Minor Linux fixes for Makefile.lite build method.

Fix all Makefile.lite to use /objs/ directory instead of /obj/ directory.

Fix building with MSYS and MinGW(-w64); Improve Makefile.lite build system

This is a patch to allow building of the project using MSYS, MinGW, and MinGW-w64 with the following invocation:

make -f Makefile.lite libFLAC libFLAC++ flac metaflac test_libs_common test_libFLAC test_libFLAC++ test_grabbag test_seeking test_streams utils examples

This patch addresses eight points:
1. `uname -p` in MSYS returns "unknown" so we must use `gcc -dumpmachine` to gain information about the target, 32-bit or 64-bit.
2. MinGW-w64 does not ship with a working iconv.h, so we must disable it under this specific compiler.
3. The code requires <inttypes.h> in a handful of C files, but config.mk did not contain -DHAVE_INTTYPES_H, which under the full build process (I assume) is added by autoconf.
4. The compiler complained when lround() in lpc.c was static, so it is no longer static.
5. Additional scattered linking directives (and reordering) (particularly FLAC, grabbag, and replaygain_analysis) were necessary to build some of the components.
6. The Makefile.lite build system benefited from some cleanup, particularly by rigorously defining all entries, factoring redundancy, and establishing dependencies. (Some typos were fixed too.)
7. Shared objects on Windows use .dll, not .so. (Added *.dll, *.dylib, and *.exe to .gitignore.)
8. To allow more freedom using Makefile.lite without configure, I added the variables USE_OGG and USE_ICONV which can toggle these two components in the build process.
ex: make -f Makefile.lite examples USE_OGG=0 USE_ICONV=0

These improvements make use of some use-time Makefile variable expansion.

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

Fix to allow the flac command to parse piped WAV input with
WAVEFORMATEXTENSIBLE format.

MinGW's fseeko() doesn't return error for the attempt to seek on non
seekable file (same behavior as MSVC).
The simplest solution would be to change #ifdef _MSC_VER to #ifdef
_WIN32 here.
Instead, this patch tests file with fstat(), and use fseeko() only when
it is a regular file.
This is confirmed to work properly both on MSVC and MinGW, can seek if
stdin is a redirected regular file, and doesn't require #ifdef.

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

Rename obj/ directory to objs/ to avoid BSD make issue.

configure.ac : Treat powerpc64 like powerpc.

configure: modernize autoconf usage

In particular:
- quote several macro arguments properly
- switch to the extended version of AC_INIT
- replace AM_CONFIG_HEADER by AC_CONFIG_HEADERS
- remove obsolete extra AC_CHECK_SIZEOF param
- upgrade minimal automake version from 1.7 to 1.11
  (which is when dist-xz was introduced)

Signed-off-by: Max Horn <max@quendi.de>

configure: merge AC_CHECK_HEADERS calls

Also removed some pointless AC_SUBST invocation, which would only be
necessary if the Makefiles were using the results of the header
detection, which they are not.

Signed-off-by: Max Horn <max@quendi.de>

configure: always print ac_cv_c_compiler_gnu

Previously, we only printed whether the used compiler was
detected as gcc if the answer was positive. Now we always
print the result (but the GCC version still is only
printed when we are using GCC)

Signed-off-by: Max Horn <max@quendi.de>

autogen.sh: replace this by a simple call to autoreconf

The autoreconf tool is provided by autoconf to do what custom
autogen.sh scripts in many projects used to do. Only it is more
robust and widely tested. It has been available for several years,
too. No reason to rely on custom code for this.

Signed-off-by: Max Horn <max@quendi.de>

configure: replace XIPH_C_FIND_ENDIAN by AC_C_BIGENDIAN

Signed-off-by: Max Horn <max@quendi.de>

Fix a copy & paste error in a comment.

Covert 2 C source files from dos to unix line endings.

Update FSF address.

Update library version-info.

Hide symbols with nasm.

Hide all cglobal symbols with nasm >= 2.

Hide symbols with gcc.

With gcc >= 4 and ELF, set default visibility to hidden and make visible
only the symbols with FLAC_API or FLACPP_API.

A convenience libFLAC-static.la is created for test_libFLAC as it
depends on the hidden symbols.

Don't override user-specified CFLAGS.

Remove old GNU-stack sections from nasm files.

They are not needed since the section is defined in nasm.h.

Add missing options to flac man page.

src/libFLAC/stream_encoder.c : Fix typo.

FLAC__treamEncoderReadStatusString -> FLAC__StreamEncoderReadStatusString

Thanks to nu774 on github.

Replace printfs with "%s" format strings with puts.

Thanks to Elan Ruusamäe for the suggestion.

Add a regresion test for compression levels.

Fix a regression that severely impacted compression levels.

Regression was introducted in commit 8bbbf56403808ff75126cd0840a936aedbc4113b
(Feb 1 19:58:09 2012) when fixing compiler warnings. In src/libFLAC/lpc.c the
line:

        ref[i] = (r/=err);

was erroniously removed because the left hand side, ref[i] was never used.
Obviously, the correct thing to do was to replace that line with:

        r /= err;

This code has not been officially released. The only people who would have
been affected are people who compiled FLAC from git between February and
now.

The only adverse affect of this error was that compression ratio would have
been severely compromised. No audio is lost, and if anyone has a file that
compressed with a bad version of FLAC can decompress it to WAV and then
re-compress with a fixed version.

Regenerate manpages.

Updates to metaflac manpage from Jaren Stangret <sirjaren@gmail.com>.

man/Makefile : Fail more gracefully.

Add a convenience function to create a Metadata::Padding object with a
certain size in one statement.

Patch from Bastiaan Timmer <basjetimmer@yahoo.com>

Optimize FLAC__bitreader_read_rice_signed_block.

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

Add new clz function which works with input 0.

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

Make FLAC__clz_soft_uint32 static.

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

Fix assembler code to make it PIC so the dynamic linker does
not have to do text relocations on-the-fly. Patch from Gentoo
via Jaren Stangret <sirjaren@gmail.com>

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

Change order of flac_LDADD libs so it works with mingw.

Add 'ar-lib' to .gitignore.

configure.ac : Make sure FLAC__HAS_OGG gets defined to either 0 or 1.

Move code inside #ifdef _MSC_VER to compat.h.

Move all definitions of FLAC__U64L to one place.

Adds last functions from C metadata api to C++ metadata api

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

Fix building when configured with --disable-shared.

The problem was that the function safe_malloc_mul_2op_() was originally
defined as static inline in inclide/share/alloc.h but had to be moved
because GCC was refusing to inline it. Once moved however, static linking
would fail when building the flac executable because the function ended
up beiong linked twice.

Fix FLAC__stream_decoder_delete() and FLAC__stream_encoder_delete()
so that they will simply return if passed a NULL pointer (ie just
like the C free() stdlib function).

Fix build with automake 1.12

recent autotools require calling AM_PROG_AR before LT_INIT
this macro needs to be conditionally defined as it is not
present in old versions.

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

Quick patch to fix FLAC_metadata_object api docs.

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

bitmath: Finish up optimizations

This patch adds support for other compilers and systems
including MSVC, Intel C compiler etc..

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

Fix more compiler warnings.

configure.ac : For gcc use -Wall -Wextra when using --enable-gcc-werror.

metadata.cpp : Fix compiler warning on 64 bit.

Patch from Bastiaan Timmer <basjetimmer@yahoo.com>.

src/libFLAC/lpc.c : Include <inttypes.h> so compiling debug version works.

Add missing functions to VorbisComment class.

Patch from Bastiaan Timmer <basjetimmer@yahoo.com> sent to the
<flac-dev@xiph.org> mailing list.

Optimize COUNT_ZERO_MSBS macro

Reorder the conditions according to the expected distribution of input
signal. This seems to make it almost as fast as the clz builtin using
the bsr instruction.

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

Fix all links to libtool version info rules.

Bump version-info for libFLAC++ from 8:0:2 to 8:0:3.

Add functionality present in the C library to the to C++ SeekTable class.

Patch from Bastiaan Timmer <basjetimmer@yahoo.com> sent to the
<flac-dev@xiph.org> mailing list.

Tweak MALLOC_PERTURB_ expression in tests.

export MALLOC_PERTURB_ and MALLOC_CHECK_ in test suite

That enables an special implementation that checks
for common memory errors and will save us from a lot pain.

replace local_strtoull with _strtoui64 in windows

Previous patch replaced the other local_strtoll with _strtoi64

Remove local_strtoull, windows has equivalent function _strtoui64

Update and improve autotools build

- INCLUDES is deprecated, and CPPFLAGS is an user-defined
  variable, use the proper AM_CPPFLAGS instead

- Remove FLAC__INLINE definition, providing proper
  replacement for MSVC compilers.

- Detect if we have C99 's lround and provide a replacement
  for windows...

Use sizeof instead of magic number 4.

Update .gitignore.

Fix make distcheck

src/libFLAC/include/private/macros.h : Add copyright header.

V2: Use a single definition of MIN and MAX in sources

remove unnecesary typedef in bitwriter.c

Fix gcc-4.7 compiler warnings.

Fix buffer overflow by replacing strcpy with memcpy.

Thanks to Cristian Rodríguez <crrodriguez@opensuse.org> for the
report.

Remove casting of return value from *alloc() functions.

configure.ac : Add AM_PROG_CC_C_O.

Remove unnecessary typedef.

Add GCC specific optimisation for log base 2 operations.

Based on a patch from Cristian Rodríguez.

Remove winamp2 plugin

Patch from Cristian Rodríguez
Nullsoft has provided a native FLAC support since Winamp 5.31 (since
the year 2006).

Add patch from Earl Chew <earl_chew@yahoo.com> that adds testing for
the replay gain calculations.

Rename ENDSWAP_INT to ENDSWAP_32.

Include stdlib.h for _response() and _wildcard() declarations on OS/2

a.out format does not support many .section directives

configure.ac : Report useful info at the end of configure.

Fix a couple 'inlining failed' warning messages.

Fix a couple of sign-compare warnings.

Port David Robinson's equalloudfilt.m MATLAB script to scilab so that the ReplayGain filter settings can be generated using an OpenSource tool.

Signed-off-by: Earl Chew <earl_chew@yahoo.com>

Support 56kHz to 19.2kHz gain analysis (Patch v4)

This implementation uses decimation to generate an estimate of the
required ReplayGain adjustment for tracks sampled at high rates.

This approach avoids having to generate filters with commensurately more taps,
and also the subsequent effect on performance as these additional
taps are evaluated for high sample rate tracks.

Filter table entries with coefficients that are unchanged are
marked /* ORIGINAL */.

The remaining entries are new and have coefficient values obtained
from src/utils/loudness/loudness.sci. See:

        http://lists.xiph.org/pipermail/flac-dev/2012-February/003220.html

Because these filter coefficients can be generated from a known source,
they are preferred to the FooBar2000 coefficients whose provenance is
unknown.

Signed-off-by: Earl Chew <earl_chew@yahoo.com>

Speed up the tests by testing slightly fewer combinations.

.gitignore : Ignore more m4 files.

Remove '#if 0'-ed out code.

Fix a bunch of -Wtype-limits warnings.

Fix -Werror=format-security errors.

Fixes "format not a string literal and no format arguments
 [-Werror=format-security]" errors.

Patch from Fabian Greffrath <fabian+debian@greffrath.com> via Debian.
Closes Debian bug #643377.

Fix for autoreconf failures.

Fixes "undefined macro: AM_PATH_XMMS" and similar errors which cause
autoreconf to fail.

From Sebastian Andrzej Siewior <bigeasy@linutronix.de> via Debian.

Disable Altivec on host triplet powerpc-unknown-linux-gnuspe.

SPE is incompatible with AltiVec because it shares the same opcode range.

Patch from Sebastian Andrzej Siewior <bigeasy@linutronix.de> via Debian.

libFLAC.m4 : Prevent empty -L flag.

Fix libFLAC.m4 may set empty -L flag when configure is called without
arguments, provided of course that the target package's configure script
is properly re-generated against the fixed libFLAC.m4

Patch from Fabian Greffrath <fabian+debian@greffrath.com> via Debian.

Merge branch 'noname'

Disable executable stack in libFLAC.so.

This allows programs using it to run correctly on SE Linux systems, and
with exec-shield.

Patch from Russell Coker <russell@coker.com.au> via Debian.

Add missing <cstring> include.

Patch from Cyril Brulebois <cyril.brulebois@enst-bretagne.fr> via
Debian. Closes Debian bug #455304.

Add missing <csting> include.

Patch from Cyril Brulebois <cyril.brulebois@enst-bretagne.fr> via
Debian. Closes Debian bug #455304.

metaflac : Exit with error if invalid option is passed.

Fix from Joshua Kwan <joshk@triplehelix.org> via Debian.
Closes Debian bug #329979.

Fix misuse of \r (potentially) in flac -d or -t output.

Patch from Joshua Kwan <joshk@triplehelix.org> via Debian.
Closes Debian bug #138828.

include/FLAC/ordinals.h : Remove CPP hackery.

This change assumes that a C99 <stdint.h> header is available. For
compilers where that is not the case, the user should provide a
minimal replacement header.