From: Michael Olbrich Date: Thu, 11 Jun 2020 06:32:05 +0000 (+0200) Subject: libs: wayland: display: only handle the first output X-Git-Tag: 1.19.3~503^2~153 X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=4b2f7a188d2fd4353585e00de35a943c5394601a;p=platform%2Fupstream%2Fgstreamer.git 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: --- diff --git a/gst-libs/gst/vaapi/gstvaapidisplay_wayland.c b/gst-libs/gst/vaapi/gstvaapidisplay_wayland.c index 660c99e..8803c29 100644 --- 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); + } } }