From 5a6f4d40515aeca16e7085805486f6216b1081a4 Mon Sep 17 00:00:00 2001 From: Sam Lantinga Date: Thu, 17 Apr 2014 20:51:28 -0700 Subject: [PATCH] Fixed bug 2485 - [PATCH] Wayland: cursor disappears permanently after window loses mouse focus Bryan Cain Using any SDL application with the Wayland backend under Weston, if the application sets a cursor with SDL_SetCursor, the cursor will work until the mouse pointer leaves the window. When the pointer re-enters the window, there will be no cursor displayed at all. I did some digging, and the reason for this is that SDL attaches the buffer to the cursor surface only once (during cursor creation) and assumes that it will stay attached. This is not how Wayland works, though - once the compositor is done rendering the buffer, it will release it, so it is no longer attached to the surface. When the cursor re-enters the window a second time, SDL sets the cursor to the same surface with no buffer attached, so no cursor is displayed. This is fixed by the attached patch, which makes SDL attach the buffer to the surface when the cursor is set, not when it is created. --- src/video/wayland/SDL_waylandmouse.c | 35 ++++++++-------------------- 1 file changed, 10 insertions(+), 25 deletions(-) diff --git a/src/video/wayland/SDL_waylandmouse.c b/src/video/wayland/SDL_waylandmouse.c index 780b44868..2590b4923 100644 --- a/src/video/wayland/SDL_waylandmouse.c +++ b/src/video/wayland/SDL_waylandmouse.c @@ -53,6 +53,7 @@ typedef struct { struct wl_surface *surface; int hot_x, hot_y; + int w, h; /* Either a preloaded cursor, or one we created ourselves */ struct wl_cursor *cursor; @@ -182,19 +183,11 @@ Wayland_CreateCursor(SDL_Surface *surface, int hot_x, int hot_y) data->surface = wl_compositor_create_surface(wd->compositor); wl_surface_set_user_data(data->surface, NULL); - wl_surface_attach(data->surface, - data->buffer, - 0, - 0); - wl_surface_damage(data->surface, - 0, - 0, - surface->w, - surface->h); - wl_surface_commit(data->surface); data->hot_x = hot_x; data->hot_y = hot_y; + data->w = surface->w; + data->h = surface->h; } return cursor; @@ -210,24 +203,13 @@ CreateCursorFromWlCursor(SDL_VideoData *d, struct wl_cursor *wlcursor) Wayland_CursorData *data = calloc (1, sizeof (Wayland_CursorData)); cursor->driverdata = (void *) data; - /* The wl_buffer here will be destroyed from wl_cursor_theme_destroy - * if we are fetching this from a wl_cursor_theme, so don't store a - * reference to it here */ - data->buffer = NULL; + data->buffer = WAYLAND_wl_cursor_image_get_buffer(wlcursor->images[0]); data->surface = wl_compositor_create_surface(d->compositor); wl_surface_set_user_data(data->surface, NULL); - wl_surface_attach(data->surface, - WAYLAND_wl_cursor_image_get_buffer(wlcursor->images[0]), - 0, - 0); - wl_surface_damage(data->surface, - 0, - 0, - wlcursor->images[0]->width, - wlcursor->images[0]->height); - wl_surface_commit(data->surface); data->hot_x = wlcursor->images[0]->hotspot_x; data->hot_y = wlcursor->images[0]->hotspot_y; + data->w = wlcursor->images[0]->width; + data->h = wlcursor->images[0]->height; data->cursor= wlcursor; } else { SDL_OutOfMemory (); @@ -315,7 +297,7 @@ Wayland_FreeCursor(SDL_Cursor *cursor) if (!d) return; - if (d->buffer) + if (d->buffer && !d->cursor) wl_buffer_destroy(d->buffer); if (d->surface) @@ -341,6 +323,9 @@ Wayland_ShowCursor(SDL_Cursor *cursor) { Wayland_CursorData *data = cursor->driverdata; + wl_surface_attach(data->surface, data->buffer, 0, 0); + wl_surface_damage(data->surface, 0, 0, data->w, data->h); + wl_surface_commit(data->surface); wl_pointer_set_cursor (pointer, 0, data->surface, data->hot_x,