From dc1c3e49bbe9ac3430a83cf2b404d75c8a3c8086 Mon Sep 17 00:00:00 2001 From: Manuel Bachmann Date: Mon, 16 Feb 2015 11:52:13 +0100 Subject: [PATCH] desktop-shell: remove dead unminimization code The "set_minimized(surface, 0)" function call was never used anywhere, and not really respecting naming conventions. Signed-off-by: Manuel Bachmann Reviewed-by: Pekka Paalanen --- 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 aca91eb8..83698e9d 100644 --- a/desktop-shell/shell.c +++ b/desktop-shell/shell.c @@ -2641,7 +2641,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; @@ -2659,31 +2659,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(¤t_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); } @@ -3927,7 +3914,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 = {