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

2013-10-11 Thread Kristian Høgsberg
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 

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@lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/wayland-devel
___
wayland-devel mailing list
wayland-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/wayland-devel


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

2013-10-10 Thread Stefan Schmidt
Hello.

Ignore the 7 in the patchset count its only these two patches.

regards
Stefan Schmidt
___
wayland-devel mailing list
wayland-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/wayland-devel