e_desk: use wl_signal_emit 80/309280/1
authorSooChan Lim <sc1.lim@samsung.com>
Mon, 8 Apr 2024 04:55:27 +0000 (13:55 +0900)
committerTizen Window System <tizen.windowsystem@gmail.com>
Mon, 8 Apr 2024 07:32:05 +0000 (16:32 +0900)
instead of wl_signal_emit_mutable to avoid the race condition on mutli-threading

Change-Id: I2fce13a6c8857fe5eba2e41aa3993724137adbef

src/bin/e_desk.c

index 5793f7e..8a2138f 100644 (file)
@@ -931,7 +931,7 @@ e_desk_client_add(E_Desk *desk, E_Client *ec)
 
    _e_desk_smart_client_add(desk->smart_obj, ec);
 
-   wl_signal_emit_mutable(&PRI(desk)->events.client_add, desk);
+   wl_signal_emit(&PRI(desk)->events.client_add, desk);
    _e_desk_hook_call(E_DESK_HOOK_CLIENT_ADD, desk);
 #else
    if (desk->desk_area.enable)
@@ -946,7 +946,7 @@ e_desk_client_add(E_Desk *desk, E_Client *ec)
 
    _e_desk_smart_client_add(desk->smart_obj, ec);
 
-   wl_signal_emit_mutable(&PRI(desk)->events.client_add, desk);
+   wl_signal_emit(&PRI(desk)->events.client_add, desk);
    _e_desk_hook_call(E_DESK_HOOK_CLIENT_ADD, desk);
 
 #endif
@@ -2129,7 +2129,7 @@ e_desk_desk_area_active_set(E_Desk *desk, E_Desk_Area *eda)
 
    desk->desk_area.active = eda;
 
-   wl_signal_emit_mutable(&PRI(desk)->events.desk_area_active_change, desk);
+   wl_signal_emit(&PRI(desk)->events.desk_area_active_change, desk);
    _e_desk_hook_call(E_DESK_HOOK_DESK_AREA_ACTIVE_CHANGE, desk);
 }
 
@@ -2176,7 +2176,7 @@ e_desk_desk_area_enable(E_Desk *desk)
    if (desk->desk_area.enable) return EINA_TRUE;
    desk->desk_area.enable = EINA_TRUE;
 
-   wl_signal_emit_mutable(&PRI(desk)->events.desk_area_enable, desk);
+   wl_signal_emit(&PRI(desk)->events.desk_area_enable, desk);
    _e_desk_hook_call(E_DESK_HOOK_DESK_AREA_ENABLE, desk);
 
    // check all ec is included in desk group
@@ -2206,7 +2206,7 @@ e_desk_desk_area_disable(E_Desk *desk)
    if (!desk->desk_area.enable) return EINA_TRUE;
    desk->desk_area.enable = EINA_FALSE;
 
-   wl_signal_emit_mutable(&PRI(desk)->events.desk_area_disable, desk);
+   wl_signal_emit(&PRI(desk)->events.desk_area_disable, desk);
    _e_desk_hook_call(E_DESK_HOOK_DESK_AREA_DISABLE, desk);
 
    for (int i=0; i<E_DESK_AREA_LAYER_COUNT; i++)
@@ -2407,7 +2407,7 @@ _e_desk_cb_split_desk_activate(struct wl_listener *listener, void *data)
   priv = container_of(listener, E_Desk_Private, ds_tizen_splitscreen_activate);
 
   // call the hook for activating the splitscreen in this zone
-  wl_signal_emit_mutable(&priv->events.split_desk_activate, NULL);
+  wl_signal_emit(&priv->events.split_desk_activate, NULL);
   _e_desk_hook_call(E_DESK_HOOK_SPLIT_DESK_ACTIVATE, priv->desk);
 }
 
@@ -2419,7 +2419,7 @@ _e_desk_cb_split_desk_deactivate(struct wl_listener *listener, void *data)
   priv = container_of(listener, E_Desk_Private, ds_tizen_splitscreen_deactivate);
 
   // call the hook for deactivating the splitscreen in this desk
-  wl_signal_emit_mutable(&priv->events.split_desk_deactivate, NULL);
+  wl_signal_emit(&priv->events.split_desk_deactivate, NULL);
   _e_desk_hook_call(E_DESK_HOOK_SPLIT_DESK_DEACTIVATE, priv->desk);
 }