From e7ffea48d937392ba8f5c2eff76a2ab0ebbb4d84 Mon Sep 17 00:00:00 2001
From: Suyeon Hwang <stom.hwang@samsung.com>
Date: Wed, 19 Jan 2022 15:05:51 +0900
Subject: [PATCH] Remove unused code about sound focus change

Change-Id: I7769384e7d16697b460a7b12af669c08b5627dc7
Signed-off-by: Suyeon Hwang <stom.hwang@samsung.com>
---
 server/AudioStream.cpp | 27 +++------------------------
 server/AudioStream.h   |  5 +----
 server/ttsd_player.cpp | 13 ++++---------
 3 files changed, 8 insertions(+), 37 deletions(-)

diff --git a/server/AudioStream.cpp b/server/AudioStream.cpp
index bd2f896..8c7b98b 100644
--- a/server/AudioStream.cpp
+++ b/server/AudioStream.cpp
@@ -23,10 +23,9 @@ using namespace std;
 static const char* FOCUS_SERVER_READY = "/tmp/.sound_server_ready";
 
 
-AudioStream::AudioStream(focusReleaseCallback focusReleaseCb)
+AudioStream::AudioStream()
 {
 	__prepared = false;
-	__focusReleaseCallback = focusReleaseCb;
 	__focusAquired = false;
 
 	createSoundStreamInfo();
@@ -59,7 +58,6 @@ void AudioStream::createSoundStreamInfo()
 
 AudioStream::~AudioStream()
 {
-	__focusReleaseCallback = nullptr;
 	__focusAquired = false;
 
 	unprepareAudioOut();
@@ -110,14 +108,14 @@ int AudioStream::acquireSoundFocus(const char* extra_info)
 
 	int ret = sound_manager_acquire_focus(__streamInfo, SOUND_STREAM_FOCUS_FOR_PLAYBACK, SOUND_BEHAVIOR_NONE, extra_info);
 	if (SOUND_MANAGER_ERROR_NONE != ret) {
-		SLOG(LOG_WARN, tts_tag(), "[AudioStream] Fail to acquire focus");
+		SLOG(LOG_WARN, tts_tag(), "[AudioStream] Fail to acquire focus. ret(%s)", get_error_message(ret));
 	} else {
 		SLOG(LOG_DEBUG, tts_tag(), "[AudioStream] Success to acquire focus");
 	}
 
 	ret = audio_out_set_sound_stream_info(__audioHandle, __streamInfo);
 	if (AUDIO_IO_ERROR_NONE != ret) {
-		SLOG(LOG_WARN, tts_tag(), "[AudioStream] Fail to set stream info");
+		SLOG(LOG_WARN, tts_tag(), "[AudioStream] Fail to set stream info. ret(%s)", get_error_message(ret));
 		return TTSD_ERROR_OPERATION_FAILED;
 	}
 
@@ -256,23 +254,4 @@ void AudioStream::__focusStateChangedCallback(sound_stream_info_h stream_info, s
 {
 	SLOG(LOG_INFO, tts_tag(), "[AudioStream] focus state changed to (%d) with reason(%d) and extra info(%s)",
 			(int)focus_state, (int)reason_for_change, extra_info);
-
-	AudioStream* audioOut = static_cast<AudioStream*>(user_data);
-	if (stream_info != audioOut->__streamInfo) {
-		SLOG(LOG_ERROR, tts_tag(), "[AudioStream] Invalid stream info handle");
-		return;
-	}
-
-	if (false == audioOut->__prepared || AUDIO_STATE_NONE == audioOut->__state || AUDIO_STATE_READY == audioOut->__state) {
-		return;
-	}
-
-	if (SOUND_STREAM_FOCUS_FOR_PLAYBACK != focus_mask || SOUND_STREAM_FOCUS_STATE_RELEASED != focus_state) {
-		return;
-	}
-
-	SLOG(LOG_INFO, tts_tag(), "[AudioStream] Focus is released on playing. Invoke callback");
-	if (audioOut->__focusReleaseCallback) {
-		audioOut->__focusReleaseCallback();
-	}
 }
diff --git a/server/AudioStream.h b/server/AudioStream.h
index a726aa4..36dcc11 100644
--- a/server/AudioStream.h
+++ b/server/AudioStream.h
@@ -31,9 +31,7 @@ public:
 		AUDIO_STATE_PLAY
 	};
 
-	typedef void (*focusReleaseCallback)();
-
-	AudioStream(focusReleaseCallback focusReleaseCb);
+	AudioStream();
 	~AudioStream();
 
 	int setAudioFormat(ttse_audio_type_e type, int rate);
@@ -63,7 +61,6 @@ private:
 	bool __prepared;
 
 	sound_stream_info_h __streamInfo;
-	focusReleaseCallback __focusReleaseCallback;
 	std::string __currentExtraInfo;
 	bool __focusAquired;
 
diff --git a/server/ttsd_player.cpp b/server/ttsd_player.cpp
index af13e0c..a8332e4 100644
--- a/server/ttsd_player.cpp
+++ b/server/ttsd_player.cpp
@@ -393,7 +393,7 @@ static void __play_utterance_cb(PlayerThread* player, unsigned int uid)
 int ttsd_player_init()
 {
 	g_background_volume = new BackgroundVolume(SND_MGR_DUCKING_DURATION);
-	g_audio_stream = new AudioStream(nullptr);
+	g_audio_stream = new AudioStream();
 	g_player_thread = new PlayerThread(__play_utterance_cb);
 
 	g_is_set_policy = false;
@@ -444,14 +444,9 @@ int ttsd_player_play(unsigned int uid)
 	}
 
 	unsigned int currentUid = g_player_thread->getCurrentUid();
-	if (0 < ttsd_data_is_client(currentUid)) {
-		if (uid == currentUid) {
-			SLOG(LOG_DEBUG, tts_tag(), "[Player] uid(%u) has already played", uid);
-			return 0;
-		} else {
-			SLOG(LOG_WARN, tts_tag(), "[Player WARNING] stop old player (%u)", currentUid);
-			ttsd_player_stop(currentUid);
-		}
+	if (0 < ttsd_data_is_client(currentUid) && uid == currentUid) {
+		SLOG(LOG_DEBUG, tts_tag(), "[Player] uid(%u) has already played", uid);
+		return TTSD_ERROR_NONE;
 	}
 
 	SLOG(LOG_INFO, tts_tag(), "[Player] start play : uid(%u)", uid);
-- 
2.7.4