wayland: Use wl_fixes for registry destruction

Author: Kontrabant

src/video/wayland/SDL_waylandvideo.c

@@ -96,6 +96,13 @@
 #define SDL_WL_OUTPUT_VERSION 3
 #endif
 
+// The SDL wayland-client minimum is 1.18, which supports version 3.
+#define SDL_WL_DATA_DEVICE_VERSION 3
+
+#if SDL_WAYLAND_CHECK_VERSION(1, 24, 0)
+#define SDL_WL_FIXES_VERSION 1
+#endif
+
 #ifdef SDL_USE_LIBDBUS
 #include "../../core/linux/SDL_dbus.h"
 
@@ -457,6 +464,7 @@ static void Wayland_DeleteDevice(SDL_VideoDevice *device)
 typedef struct
 {
     bool has_fifo_v1;
+    struct wl_fixes *wl_fixes;
 } SDL_WaylandPreferredData;
 
 static void wayland_preferred_check_handle_global(void *data, struct wl_registry *registry, uint32_t id,
@@ -466,6 +474,10 @@ static void wayland_preferred_check_handle_global(void *data, struct wl_registry
 
     if (SDL_strcmp(interface, "wp_fifo_manager_v1") == 0) {
         d->has_fifo_v1 = true;
+#ifdef SDL_WL_FIXES_VERSION
+    } else if (SDL_strcmp(interface, "wl_fixes") == 0) {
+        d->wl_fixes = wl_registry_bind(registry, id, &wl_fixes_interface, SDL_min(SDL_WL_FIXES_VERSION, version));
+#endif
     }
 }
 
@@ -493,6 +505,12 @@ static bool Wayland_IsPreferred(struct wl_display *display)
 
     WAYLAND_wl_display_roundtrip(display);
 
+#ifdef WL_FIXES_INTERFACE
+    if (preferred_data.wl_fixes) {
+        wl_fixes_destroy_registry(preferred_data.wl_fixes, registry);
+        wl_fixes_destroy(preferred_data.wl_fixes);
+    }
+#endif
     wl_registry_destroy(registry);
 
     if (!preferred_data.has_fifo_v1) {
@@ -1313,6 +1331,10 @@ static void display_handle_global(void *data, struct wl_registry *registry, uint
     } else if (SDL_strcmp(interface, "wp_color_manager_v1") == 0) {
         d->wp_color_manager_v1 = wl_registry_bind(d->registry, id, &wp_color_manager_v1_interface, 1);
         Wayland_InitColorManager(d);
+#ifdef SDL_WL_FIXES_VERSION
+    } else if (SDL_strcmp(interface, "wl_fixes") == 0) {
+        d->wl_fixes = wl_registry_bind(d->registry, id, &wl_fixes_interface, SDL_min(SDL_WL_FIXES_VERSION, version));
+#endif
     }
 }
 
@@ -1628,6 +1650,13 @@ static void Wayland_VideoCleanup(SDL_VideoDevice *_this)
     }
 
     if (data->registry) {
+#ifdef SDL_WL_FIXES_VERSION
+        if (data->wl_fixes) {
+            wl_fixes_destroy_registry(data->wl_fixes, data->registry);
+            wl_fixes_destroy(data->wl_fixes);
+            data->wl_fixes = NULL;
+        }
+#endif
         wl_registry_destroy(data->registry);
         data->registry = NULL;
     }

src/video/wayland/SDL_waylandvideo.h

@@ -84,6 +84,7 @@ struct SDL_VideoData
     struct frog_color_management_factory_v1 *frog_color_management_factory_v1;
     struct wp_color_manager_v1 *wp_color_manager_v1;
     struct zwp_tablet_manager_v2 *tablet_manager;
+    struct wl_fixes *wl_fixes;
 
     struct xkb_context *xkb_context;