From: Manuel Bachmann <[email protected]>

The "set_minimized(surface, 0)" function call was never
used anywhere, and not really respecting naming
conventions.

Signed-off-by: Manuel Bachmann <[email protected]>
---
 desktop-shell/shell.c |   33 ++++++++++-----------------------
 1 file changed, 10 insertions(+), 23 deletions(-)

diff --git a/desktop-shell/shell.c b/desktop-shell/shell.c
index f28fc10..5900c4d 100644
--- a/desktop-shell/shell.c
+++ b/desktop-shell/shell.c
@@ -2637,7 +2637,7 @@ unset_maximized(struct shell_surface *shsurf)
 }
 
 static void
-set_minimized(struct weston_surface *surface, uint32_t is_true)
+set_minimized(struct weston_surface *surface)
 {
        struct shell_surface *shsurf;
        struct workspace *current_ws;
@@ -2655,31 +2655,18 @@ set_minimized(struct weston_surface *surface, uint32_t 
is_true)
        current_ws = get_current_workspace(shsurf->shell);
 
        weston_layer_entry_remove(&view->layer_link);
-        /* hide or show, depending on the state */
-       if (is_true) {
-               
weston_layer_entry_insert(&shsurf->shell->minimized_layer.view_list, 
&view->layer_link);
-
-               drop_focus_state(shsurf->shell, current_ws, view->surface);
-               wl_list_for_each(seat, &shsurf->shell->compositor->seat_list, 
link) {
-                       if (!seat->keyboard)
-                               continue;
-                       focus = 
weston_surface_get_main_surface(seat->keyboard->focus);
-                       if (focus == view->surface)
-                               weston_keyboard_set_focus(seat->keyboard, NULL);
-               }
-       }
-       else {
-               weston_layer_entry_insert(&current_ws->layer.view_list, 
&view->layer_link);
+       weston_layer_entry_insert(&shsurf->shell->minimized_layer.view_list, 
&view->layer_link);
 
-               wl_list_for_each(seat, &shsurf->shell->compositor->seat_list, 
link) {
-                       if (!seat->keyboard)
-                               continue;
-                       activate(shsurf->shell, view->surface, seat, true);
-               }
+       drop_focus_state(shsurf->shell, current_ws, view->surface);
+       wl_list_for_each(seat, &shsurf->shell->compositor->seat_list, link) {
+               if (!seat->keyboard)
+                       continue;
+               focus = weston_surface_get_main_surface(seat->keyboard->focus);
+               if (focus == view->surface)
+                       weston_keyboard_set_focus(seat->keyboard, NULL);
        }
 
        shell_surface_update_child_surface_layers(shsurf);
-
        weston_view_damage_below(view);
 }
 
@@ -3876,7 +3863,7 @@ xdg_surface_set_minimized(struct wl_client *client,
                return;
 
         /* apply compositor's own minimization logic (hide) */
-       set_minimized(shsurf->surface, 1);
+       set_minimized(shsurf->surface);
 }
 
 static const struct xdg_surface_interface xdg_surface_implementation = {
-- 
1.7.10.4

_______________________________________________
wayland-devel mailing list
[email protected]
http://lists.freedesktop.org/mailman/listinfo/wayland-devel

Reply via email to