[PATCH 1/7] xwayland: Remove unused call weston_wm_handle_configure_notify.

Kristian Høgsberg hoegsberg at gmail.com
Fri Oct 11 22:31:12 CEST 2013


On Thu, Oct 10, 2013 at 02:28:29PM +0100, Stefan Schmidt wrote:
> We get the child position but never use this information here. Just remove it.
> Spotted by Christopher Michael.
> 
> Signed-off-by: Stefan Schmidt <s.schmidt at samsung.com>

Oh yeah, that looks unecessary, thanks.

Kristian

> ---
>  src/xwayland/window-manager.c |    2 --
>  1 file changed, 2 deletions(-)
> 
> diff --git a/src/xwayland/window-manager.c b/src/xwayland/window-manager.c
> index 2c1c9e0..ef59e6d 100644
> --- a/src/xwayland/window-manager.c
> +++ b/src/xwayland/window-manager.c
> @@ -609,7 +609,6 @@ weston_wm_handle_configure_notify(struct weston_wm *wm, xcb_generic_event_t *eve
>  	xcb_configure_notify_event_t *configure_notify = 
>  		(xcb_configure_notify_event_t *) event;
>  	struct weston_wm_window *window;
> -	int x, y;
>  
>  	wm_log("XCB_CONFIGURE_NOTIFY (window %d) %d,%d @ %dx%d\n",
>  	       configure_notify->window,
> @@ -617,7 +616,6 @@ weston_wm_handle_configure_notify(struct weston_wm *wm, xcb_generic_event_t *eve
>  	       configure_notify->width, configure_notify->height);
>  
>  	window = hash_table_lookup(wm->window_hash, configure_notify->window);
> -	weston_wm_window_get_child_position(window, &x, &y);
>  	window->x = configure_notify->x;
>  	window->y = configure_notify->y;
>  	if (window->override_redirect) {
> -- 
> 1.7.9.5
> 
> _______________________________________________
> wayland-devel mailing list
> wayland-devel at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/wayland-devel


More information about the wayland-devel mailing list