From: Suyeon Hwang <stom.hwang@samsung.com>
Date: Thu, 30 Nov 2023 10:03:20 +0000 (+0900)
Subject: Remove unnecessary sub thread checking logic
X-Git-Tag: accepted/tizen/7.0/unified/20240111.114549~5
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=aa8f895e8e22968d736bd842ecdd3ec8328a2b88;p=platform%2Fcore%2Fuifw%2Ftts.git

Remove unnecessary sub thread checking logic

- Issue:
Sometimes tts_destroy() is delayed without any reason.

- Solution:
This patch removes unncessary logic for checking active ecore sub
thread. In old version, tts.c module makes a sub thread for some reason
so the framework should check the active subthread for safe handle
destroy. However, since tizen 7.0, tts.c module does not make a sub
thread any more. And only the tts_core.c makes some sub thread and those
sub threads are completely controlled by the framework. So the checking
logic in tts.c is unncessary now.

Change-Id: I5f2ea8aa16f65e82e94332c046ae74ffba569f78
Signed-off-by: Suyeon Hwang <stom.hwang@samsung.com>
---

diff --git a/client/tts.c b/client/tts.c
index 191d2d46..3cc43f11 100644
--- a/client/tts.c
+++ b/client/tts.c
@@ -328,17 +328,6 @@ static int destroy_tts_handle(tts_h tts)
 		/* Unset registered callbacks */
 		tts_client_unset_all_cb(client);
 
-		int thread_count = ecore_thread_active_get();
-		SLOG(LOG_INFO, TAG_TTSC, "[INFO] Active thread count: %d", thread_count);
-		for (int cnt = 0; 0 < thread_count; cnt++) {
-			usleep(50000);
-			if (30 == cnt) {
-				SLOG(LOG_WARN, TAG_TTSC, "[WARNNING] Thread is blocked, %d", thread_count);
-				break;
-			}
-			thread_count = ecore_thread_active_get();
-		}
-
 		if (0 != tts_ipc_close_connection(uid)) {
 			SLOG(LOG_ERROR, TAG_TTSC, "[ERROR] Fail to close connection");
 			return TTS_ERROR_OPERATION_FAILED;