[893/906] x11: close both of the display's we use

(the comment doesn't seem to apply anymore)

https://bugzilla.gnome.org/show_bug.cgi?id=725048
This commit is contained in:
Matthew Waters 2014-02-24 23:55:58 +11:00 committed by Tim-Philipp Müller
parent d169aa05dc
commit 8f29fffc2f

View file

@ -292,19 +292,9 @@ gst_gl_window_x11_close (GstGLWindow * window)
while (XPending (window_x11->device)) while (XPending (window_x11->device))
XNextEvent (window_x11->device, &event); XNextEvent (window_x11->device, &event);
/*XAddToSaveSet (display, w) XCloseDisplay (window_x11->device);
Display *display;
Window w; */
//FIXME: it seems it causes destroy all created windows, even by other display connection:
//This is case in: gst-launch-0.10 videotestsrc ! tee name=t t. ! queue ! glimagesink t. ! queue ! glimagesink
//When the first window is closed and so its display is closed by the following line, then the other Window managed by the
//other glimagesink, is not useable and so each opengl call causes a segmentation fault.
//Maybe the solution is to use: XAddToSaveSet
//The following line is commented to avoid the disagreement explained before.
//XCloseDisplay (window_x11->device);
GST_DEBUG ("display receiver closed"); GST_DEBUG ("display receiver closed");
XCloseDisplay (window_x11->disp_send); XCloseDisplay (window_x11->disp_send);
GST_DEBUG ("display sender closed"); GST_DEBUG ("display sender closed");
} }