release v0.18.4

Just an attempt to prevent users from shooting themselves in the foot
by uncommenting these lines without knowing what they're doing.

NEWS

@@ -1,3 +1,9 @@
+ver 0.18.4 (2013/11/13)
+* decoder
+  - dsdiff: fix byte order bug
+* fix build failures due to missing includes
+* libc++ compatibility
+
 ver 0.18.3 (2013/11/08)
 * fix stuck MPD after song change (0.18.2 regression)
 

configure.ac

@@ -1,6 +1,6 @@
 AC_PREREQ(2.60)
 
-AC_INIT(mpd, 0.18.3, musicpd-dev-team@lists.sourceforge.net)
+AC_INIT(mpd, 0.18.4, musicpd-dev-team@lists.sourceforge.net)
 
 VERSION_MAJOR=0
 VERSION_MINOR=18

doc/mpdconf.example

@@ -210,7 +210,6 @@ input {
 #	type		"alsa"
 #	name		"My ALSA Device"
 ##	device		"hw:0,0"	# optional
-##	format		"44100:16:2"	# optional
 ##	mixer_type      "hardware"	# optional
 ##	mixer_device	"default"	# optional
 ##	mixer_control	"PCM"		# optional
@@ -223,7 +222,6 @@ input {
 #	type		"oss"
 #	name		"My OSS Device"
 ##	device		"/dev/dsp"	# optional
-##	format		"44100:16:2"	# optional
 ##	mixer_type      "hardware"	# optional
 ##	mixer_device	"/dev/mixer"	# optional
 ##	mixer_control	"PCM"		# optional
@@ -327,12 +325,6 @@ input {
 #	mixer_type      "none"			# optional
 #}
 #
-# This setting will change all decoded audio to be converted to the specified
-# format before being passed to the audio outputs. By default, this setting is
-# disabled.
-#
-#audio_output_format		"44100:16:2"
-#
 # If MPD has been compiled with libsamplerate support, this setting specifies 
 # the sample rate converter to use.  Possible values can be found in the 
 # mpd.conf man page or the libsamplerate documentation. By default, this is

src/DecoderAPI.cxx

@@ -35,6 +35,7 @@
 #include <assert.h>
 #include <stdlib.h>
 #include <string.h>
+#include <math.h>
 
 void
 decoder_initialized(Decoder &decoder,

src/DecoderThread.cxx

@@ -38,6 +38,8 @@
 #include "tag/ApeReplayGain.hxx"
 #include "Log.hxx"
 
+#include <functional>
+
 static constexpr Domain decoder_thread_domain("decoder_thread");
 
 /**

src/DespotifyUtils.cxx

@@ -28,6 +28,8 @@ extern "C" {
 #include <despotify.h>
 }
 
+#include <stdio.h>
+
 const Domain despotify_domain("despotify");
 
 static struct despotify_session *g_session;

src/SongSave.cxx

@@ -30,6 +30,7 @@
 #include "util/Domain.hxx"
 
 #include <string.h>
+#include <stdlib.h>
 
 #define SONG_MTIME "mtime"
 #define SONG_END "song_end"

src/decoder/DsdLib.hxx

@@ -47,6 +47,17 @@ public:
 	}
 };
 
+class DffDsdUint64 {
+	uint32_t hi;
+	uint32_t lo;
+
+public:
+	constexpr uint64_t Read() const {
+		return (uint64_t(FromBE32(hi)) << 32) |
+			uint64_t(FromBE32(lo));
+	}
+};
+
 bool
 dsdlib_read(Decoder *decoder, InputStream &is,
 	    void *data, size_t length);

src/decoder/DsdiffDecoderPlugin.cxx

@@ -43,13 +43,13 @@
 
 struct DsdiffHeader {
 	DsdId id;
-	DsdUint64 size;
+	DffDsdUint64 size;
 	DsdId format;
 };
 
 struct DsdiffChunkHeader {
 	DsdId id;
-	DsdUint64 size;
+	DffDsdUint64 size;
 
 	/**
 	 * Read the "size" attribute from the specified header, converting it

src/encoder/OpusEncoderPlugin.cxx

@@ -34,6 +34,7 @@
 #include <glib.h>
 
 #include <assert.h>
+#include <stdlib.h>
 
 struct opus_encoder {
 	/** the base class */

src/filter/RouteFilterPlugin.cxx

@@ -56,6 +56,7 @@
 #include <assert.h>
 #include <string.h>
 #include <stdint.h>
+#include <stdlib.h>
 
 class RouteFilter final : public Filter {
 	/**

src/output/HttpdInternal.hxx

@@ -30,6 +30,11 @@
 #include "thread/Mutex.hxx"
 #include "event/ServerSocket.hxx"
 
+#ifdef _LIBCPP_VERSION
+/* can't use incomplete template arguments with libc++ */
+#include "HttpdClient.hxx"
+#endif
+
 #include <forward_list>
 
 struct config_param;

src/util/LazyRandomEngine.hxx

@@ -49,12 +49,12 @@ public:
 	 */
 	void AutoCreate();
 
-	result_type min() const {
-		return engine->min();
+	static constexpr result_type min() {
+		return std::mt19937::min();
 	}
 
-	result_type max() const {
-		return engine->max();
+	static constexpr result_type max() {
+		return std::mt19937::max();
 	}
 
 	result_type operator()() {

src/util/RefCount.hxx

@@ -42,7 +42,12 @@ class RefCount {
 	std::atomic_uint n;
 
 public:
-	constexpr RefCount():n(1) {}
+#ifndef _LIBCPP_VERSION
+	/* the "constexpr" is missing in libc++'s "atomic"
+	   implementation */
+	constexpr
+#endif
+	RefCount():n(1) {}
 
 	void Increment() {
 		++n;

test/run_convert.cxx

@@ -36,6 +36,7 @@
 
 #include <assert.h>
 #include <stddef.h>
+#include <stdlib.h>
 #include <unistd.h>
 
 static void