Ulrich Klauer authored May 25, 2013 and Erik de Castro Lopo committed May 26, 2013

Use Benjamin Stiglitz' MIN macros from gcc 4.3 (according to the
changelog, __COUNTER__ was introduced in this version). Previously,
the macros weren't used on any existing gcc version; the first one
would have been 5.5.

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

Robert Kausch authored May 25, 2013 and Erik de Castro Lopo committed May 26, 2013

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

Robert Kausch authored May 25, 2013 and Erik de Castro Lopo committed May 26, 2013

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

Ulrich Klauer authored May 25, 2013 and Erik de Castro Lopo committed May 26, 2013

Commits a7e3705d and
a4c321e4

, while trying to simplify how
the FLAC_API_SUPPORTS_OGG_FLAC global variable was initialized,
inadvertently caused it to be always set to false, whether Ogg support
was compiled in or not.

This commit reverts the relevant part to how it looked in the 1.2.1
release, which is verbose but correct.

The problem was found by Robert Kausch <robert.kausch@freac.org>.

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

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

Martijn van Beurden authored May 15, 2013 and Erik de Castro Lopo committed May 25, 2013

This fixes a wrong link (dead anchor) in the Doxygen api documentation

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

Ulrich Klauer authored Apr 30, 2013 and Erik de Castro Lopo committed May 01, 2013

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

Ulrich Klauer authored Apr 11, 2013 and Erik de Castro Lopo committed Apr 30, 2013

In the flac(1) man page, fix the path to the HTML documentation. Patch
by Joshua Kwan <joshk@triplehelix.org>, taken from the Debian patch
tracker for flac 1.2.1-6 (02_doc_path.patch).

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

Ulrich Klauer authored Apr 11, 2013 and Erik de Castro Lopo committed Apr 30, 2013

When a locale is in effect that does not use the point as the decimal
mark (e.g., sv_SE or de_DE, which use a comma) and a ReplayGain tag is
read for --apply-replaygain-which-is-not-lossless, the gain value was
misinterpreted (e.g., "-2.29" truncated to "-2"). This is fixed by
resetting the locale to "C" temporarily, based on Josh Coalson's fix
of the dual case (writing ReplayGain tag) in commit cda02d30.

Patch by hhaamu@gmail.com, taken from the Debian patch tracker for
flac 1.2.1-6 (13_replaygain_c_locale.patch).

http://sourceforge.net/p/flac/bugs/380/



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

Patch from Dave Yeo <dave.r.yeo@gmail.com>.

Also regenerate man/flac.1.

Martijn van Beurden authored Apr 25, 2013 and Erik de Castro Lopo committed Apr 26, 2013

test_metaflac.sh wasn't aborting when replaygain values were
incorrect because the die() function was embedded in a pipe.
set -e was added so the script exits with a non-zero exit code
when die is called in a pipe.

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

The embedded awk script was using the 'lshift' function which
apparently only exists in GNU auk (gawk) and definitely does not
exist in mawk.

Thanks to Martijn van Beurden <mvanb1@gmail.com> for reporting
this issue.

$RANDOM seems to be a bashism and isn't avaiable in eg dash.

Janne Hyvärinen authored Apr 25, 2013 and Erik de Castro Lopo committed Apr 25, 2013

Metaflac can now print all console supported characters from tags on the
screen. It also fixes metaflac to be able to import its own exports back
without non-ascii characters getting mutilated. And --no-utf8-convert
now works properly with import and export commands.

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

Martijn van Beurden authored Apr 21, 2013 and Erik de Castro Lopo committed Apr 22, 2013

My Valgrind doesn't run, saying it doesn't support showing more
than 50 entries of a stack trace

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

Martijn van Beurden authored Apr 21, 2013 and Erik de Castro Lopo committed Apr 22, 2013

This should enable using the disable-thorough-tests,
enable-exhaustive-tests and enable-valgrind-testing
configure switches, because setting these didn't do
anything

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

Janne Hyvärinen authored Apr 21, 2013 and Erik de Castro Lopo committed Apr 21, 2013

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

Problem reported by Martijn van Beurden <mvanb1@gmail.com>.

Change metaflac hexdump function so utf-8 decoding is only used for
filename printing and changed hex output printing to not rely only
on isprint. That function seems to return true for tabulator
control character under Windows when application isn't using C-locale.

Patch (with one minor tweak) from Janne Hyvärinen <cse@sci.fi>.

Janne Hyvärinen authored Apr 20, 2013 and Erik de Castro Lopo committed Apr 21, 2013

UTF-8 version of utime was completely broken and file timestamps were
not preserved.

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

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

Cristian Rodríguez authored Apr 18, 2013 and Erik de Castro Lopo committed Apr 21, 2013

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

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

Problem reported by Ulrich Klauer <ulrich@chirlu.de>.
Patch from Janne Hyvärinen <cse@sci.fi>.