gl/window/x11: don't create our own X11 display
authorMatthew Waters <matthew@centricular.com>
Thu, 11 Jun 2015 14:32:00 +0000 (00:32 +1000)
committerMatthew Waters <matthew@centricular.com>
Thu, 11 Jun 2015 14:34:58 +0000 (00:34 +1000)
It's not needed anymore with most window operations occuring in the
GL thread.

gst-libs/gst/gl/x11/gstglwindow_x11.c

index 30caec53eb826e8519ce3e36a3a715cbc368a497..6291224a130e2efaebe17f8124826b7bf226ca46 100644 (file)
@@ -149,8 +149,7 @@ gst_gl_window_x11_open (GstGLWindow * window, GError ** error)
   GstGLWindowX11 *window_x11 = GST_GL_WINDOW_X11 (window);
   GstGLDisplayX11 *display_x11 = (GstGLDisplayX11 *) window->display;
 
-  window_x11->device = XOpenDisplay (display_x11->name);
-//  window_x11->device = display_x11->display;
+  window_x11->device = display_x11->display;
   if (window_x11->device == NULL) {
     g_set_error (error, GST_GL_WINDOW_ERROR,
         GST_GL_WINDOW_ERROR_RESOURCE_UNAVAILABLE,
@@ -158,8 +157,6 @@ gst_gl_window_x11_open (GstGLWindow * window, GError ** error)
     goto failure;
   }
 
-  XSynchronize (window_x11->device, FALSE);
-
   GST_LOG ("gl device id: %ld", (gulong) window_x11->device);
 
   window_x11->screen = DefaultScreenOfDisplay (window_x11->device);
@@ -274,13 +271,8 @@ void
 gst_gl_window_x11_close (GstGLWindow * window)
 {
   GstGLWindowX11 *window_x11 = GST_GL_WINDOW_X11 (window);
-  GstGLDisplay *display = window->display;
-  XEvent event;
 
   if (window_x11->device) {
-    /* Avoid BadDrawable Errors... */
-    if (gst_gl_display_get_handle_type (display) & GST_GL_DISPLAY_TYPE_X11)
-      XSync (GST_GL_DISPLAY_X11 (display)->display, FALSE);
 
     if (window_x11->internal_win_id)
       XUnmapWindow (window_x11->device, window_x11->internal_win_id);
@@ -292,12 +284,7 @@ gst_gl_window_x11_close (GstGLWindow * window)
           window_x11->root, 0, 0);
       XDestroyWindow (window_x11->device, window_x11->internal_win_id);
     }
-    XSync (window_x11->device, FALSE);
-
-    while (XPending (window_x11->device))
-      XNextEvent (window_x11->device, &event);
 
-    XCloseDisplay (window_x11->device);
     GST_DEBUG ("display receiver closed");
   }