[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