Remove unnecessary using idler

In the cynara module, calling g_idle_add() with the RequestCb() is not
needed. This patch removes that.

Change-Id: Ic1c8ef8a418801cce3dfb1b3731ae430b2c4a475
Signed-off-by: Hwankyu Jhun <h.jhun@samsung.com>

diff --git a/src/modules/cynara-core/cynara_manager.cc b/src/modules/cynara-core/cynara_manager.cc
index 84858cb..0531f94 100644 (file)
--- a/src/modules/cynara-core/cynara_manager.cc
+++ b/src/modules/cynara-core/cynara_manager.cc
@@ -289,8 +289,9 @@ int CynaraManager::CheckAsync(amd_request_h req,
     return ret;
   }
 
-  g_idle_add(RequestCb, cynara_request);
   res_cbs_[req_id] = std::make_shared<CynaraResponseCb>(req_id, callback, req);
+  CynaraManager::GetInst().SendRequest(
+      std::shared_ptr<CynaraRequest>(cynara_request));
   return AMD_CYNARA_RET_UNKNOWN;
 }
 
@@ -455,13 +456,6 @@ void CynaraManager::CynaraThread() {
   _W("END");
 }
 
-gboolean CynaraManager::RequestCb(gpointer user_data) {
-  auto* request = static_cast<CynaraRequest*>(user_data);
-  _W("tag: %s", request->GetTag().c_str());
-  CynaraManager::GetInst().SendRequest(std::shared_ptr<CynaraRequest>(request));
-  return G_SOURCE_REMOVE;
-}
-
 gboolean CynaraManager::ResponseCb(gpointer user_data) {
   auto* response = static_cast<CynaraResponse*>(user_data);
   CynaraManager::GetInst().DeliverResponse(

diff --git a/src/modules/cynara-core/cynara_manager.hh b/src/modules/cynara-core/cynara_manager.hh
index d207d30..7f61eba 100644 (file)
--- a/src/modules/cynara-core/cynara_manager.hh
+++ b/src/modules/cynara-core/cynara_manager.hh
@@ -59,7 +59,6 @@ class CynaraManager {
   void SendRequest(std::shared_ptr<CynaraRequest> request);
   void DeliverResponse(std::shared_ptr<CynaraResponse> response);
   void CynaraThread();
-  static gboolean RequestCb(gpointer user_data);
   static gboolean ResponseCb(gpointer user_data);
   void SetBoostingTimer();
   void UnsetBoostingTimer();