libs: wayland: display: only handle the first output

Right now, all outputs are handled. The means that the registry object for
all but the last are leaked. As a result the sizes are not used correctly.

With two outputs, at first the mode and physical size of the second output
are used. If the first output changes the mode, then the physical size of
the second output is used in combination with the resolution of the first
output. The resulting pixel aspect ratio is incorrect.

There seems to be no way to determine on which output the window is shown,
so just use the first one to get consistent results.

Part-of: <https://gitlab.freedesktop.org/gstreamer/gstreamer-vaapi/-/merge_requests/341>

diff --git a/gst-libs/gst/vaapi/gstvaapidisplay_wayland.c b/gst-libs/gst/vaapi/gstvaapidisplay_wayland.c
index 660c99e..8803c29 100644 (file)
--- a/gst-libs/gst/vaapi/gstvaapidisplay_wayland.c
+++ b/gst-libs/gst/vaapi/gstvaapidisplay_wayland.c
@@ -122,8 +122,10 @@ registry_handle_global (void *data,
         wl_registry_bind (registry, id, &xdg_wm_base_interface, 1);
     xdg_wm_base_add_listener (priv->xdg_wm_base, &xdg_wm_base_listener, priv);
   } else if (strcmp (interface, "wl_output") == 0) {
-    priv->output = wl_registry_bind (registry, id, &wl_output_interface, 1);
-    wl_output_add_listener (priv->output, &output_listener, priv);
+    if (!priv->output) {
+      priv->output = wl_registry_bind (registry, id, &wl_output_interface, 1);
+      wl_output_add_listener (priv->output, &output_listener, priv);
+    }
   }
 }