From 7f16eb12453b35a427ddd1551615bbbc66ac1f10 Mon Sep 17 00:00:00 2001 From: Suyeon Hwang Date: Thu, 30 Nov 2023 19:03:20 +0900 Subject: [PATCH] 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 --- client/tts.c | 11 ----------- 1 file changed, 11 deletions(-) diff --git a/client/tts.c b/client/tts.c index c47ef85..5bb7037 100644 --- a/client/tts.c +++ b/client/tts.c @@ -299,17 +299,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; -- 2.7.4