From 917cedf893ee73317ed191f95ab201656f0d5b30 Mon Sep 17 00:00:00 2001
From: Max Kellermann <max@musicpd.org>
Date: Sun, 15 Jan 2017 01:23:49 +0100
Subject: [PATCH] output/Thread: move AudioFormat logging code around

---
 src/output/OutputThread.cxx | 26 +++++++++++++++-----------
 1 file changed, 15 insertions(+), 11 deletions(-)

diff --git a/src/output/OutputThread.cxx b/src/output/OutputThread.cxx
index 67d321c55..809cae6cc 100644
--- a/src/output/OutputThread.cxx
+++ b/src/output/OutputThread.cxx
@@ -141,6 +141,15 @@ AudioOutput::Open()
 
 		open = true;
 	}
+
+	if (f != source.GetInputAudioFormat() || f != out_audio_format) {
+		struct audio_format_string afs1, afs2, afs3;
+		FormatDebug(output_domain, "converting in=%s -> f=%s -> out=%s",
+			    audio_format_to_string(source.GetInputAudioFormat(),
+						   &afs1),
+			    audio_format_to_string(f, &afs2),
+			    audio_format_to_string(out_audio_format, &afs3));
+	}
 }
 
 void
@@ -155,6 +164,12 @@ AudioOutput::OpenOutputAndConvert(AudioFormat desired_audio_format)
 							  name, plugin.name));
 	}
 
+	struct audio_format_string af_string;
+	FormatDebug(output_domain,
+		    "opened plugin=%s name=\"%s\" audio_format=%s",
+		    plugin.name, name,
+		    audio_format_to_string(out_audio_format, &af_string));
+
 	try {
 		convert_filter_set(convert_filter.Get(), out_audio_format);
 	} catch (const std::runtime_error &e) {
@@ -178,17 +193,6 @@ AudioOutput::OpenOutputAndConvert(AudioFormat desired_audio_format)
 		std::throw_with_nested(FormatRuntimeError("Failed to convert for \"%s\" [%s]",
 							  name, plugin.name));
 	}
-
-	struct audio_format_string af_string;
-	FormatDebug(output_domain,
-		    "opened plugin=%s name=\"%s\" audio_format=%s",
-		    plugin.name, name,
-		    audio_format_to_string(out_audio_format, &af_string));
-
-	if (source.GetInputAudioFormat() != out_audio_format)
-		FormatDebug(output_domain, "converting from %s",
-			    audio_format_to_string(source.GetInputAudioFormat(),
-						   &af_string));
 }
 
 void