[PATCH weston v2 16/24] xwm: delete dead flags from weston_wm_window_draw_decorations()

Quentin Glidic sardemff7+wayland at sardemff7.net
Sun Jan 15 13:59:08 UTC 2017


On 21/12/2016 15:40, Pekka Paalanen wrote:
> From: Pekka Paalanen <pekka.paalanen at collabora.co.uk>
> 
> Obviously unused. Looks like weston_wm_window_activate() is doing that
> job.
> 
> Signed-off-by: Pekka Paalanen <pekka.paalanen at collabora.co.uk>

Trivial:
Reviewed-by: Quentin Glidic <sardemff7+git at sardemff7.net>

Thanks,


> ---
>   xwayland/window-manager.c | 4 ----
>   1 file changed, 4 deletions(-)
> 
> diff --git a/xwayland/window-manager.c b/xwayland/window-manager.c
> index 021b18a..9071598 100644
> --- a/xwayland/window-manager.c
> +++ b/xwayland/window-manager.c
> @@ -1129,7 +1129,6 @@ weston_wm_window_draw_decoration(void *data)
>   	int32_t input_x, input_y, input_w, input_h;
>   	const struct weston_desktop_xwayland_interface *xwayland_interface =
>   		wm->server->compositor->xwayland_interface;
> -	uint32_t flags = 0;
>   
>   	wm_log("XWM: start draw decoration, win %d\n", window->id);
>   
> @@ -1146,9 +1145,6 @@ weston_wm_window_draw_decoration(void *data)
>   	if (window->fullscreen) {
>   		/* nothing */
>   	} else if (window->decorate) {
> -		if (wm->focus_window == window)
> -			flags |= THEME_FRAME_ACTIVE;
> -
>   		frame_set_title(window->frame, window->name);
>   		frame_repaint(window->frame, cr);
>   	} else {
> 


-- 

Quentin “Sardem FF7” Glidic


More information about the wayland-devel mailing list