Don’t initialize the AUGraph until after setting AudioUnit parameters

sbooth · Aug 14, 2013 · 63c40be · 63c40be
1 parent 0d502ce
commit 63c40be
Showing 1 changed file with 13 additions and 13 deletions.
diff --git a/Player/AudioPlayer.cpp b/Player/AudioPlayer.cpp
@@ -2226,20 +2226,7 @@ bool AudioPlayer::OpenOutput()
 		mAUGraph = nullptr;
 		return false;
 	}
-
-	// Initialize the graph
-	result = AUGraphInitialize(mAUGraph);
-	if(noErr != result) {
-		LOGGER_ERR("org.sbooth.AudioEngine.AudioPlayer", "AUGraphInitialize failed: " << result);
 
-		result = DisposeAUGraph(mAUGraph);
-		if(noErr != result)
-			LOGGER_ERR("org.sbooth.AudioEngine.AudioPlayer", "DisposeAUGraph failed: " << result);
-
-		mAUGraph = nullptr;
-		return false;
-	}
-
 	// Set the mixer's volume on the input and output
 	AudioUnit au = nullptr;
 	result = AUGraphNodeInfo(mAUGraph, mMixerNode, nullptr, &au);
@@ -2330,6 +2317,19 @@ bool AudioPlayer::OpenOutput()
 	}
 #endif
 
+	// Initialize the graph
+	result = AUGraphInitialize(mAUGraph);
+	if(noErr != result) {
+		LOGGER_ERR("org.sbooth.AudioEngine.AudioPlayer", "AUGraphInitialize failed: " << result);
+
+		result = DisposeAUGraph(mAUGraph);
+		if(noErr != result)
+			LOGGER_ERR("org.sbooth.AudioEngine.AudioPlayer", "DisposeAUGraph failed: " << result);
+
+		mAUGraph = nullptr;
+		return false;
+	}
+
 	return true;
 }