From 46decac44efd74ced7b5ea9c9556d12b14b051b5 Mon Sep 17 00:00:00 2001
From: Matthew Waters <ystreet00@gmail.com>
Date: Sun, 2 Jun 2013 21:47:17 +1000
Subject: [PATCH] [690/906] windowx11glx: use fbconfigs if supported

---
 gst-libs/gst/gl/x11/gstglwindow_x11_glx.c | 89 +++++++++++++++++++----
 gst-libs/gst/gl/x11/gstglwindow_x11_glx.h |  5 +-
 2 files changed, 80 insertions(+), 14 deletions(-)

diff --git a/gst-libs/gst/gl/x11/gstglwindow_x11_glx.c b/gst-libs/gst/gl/x11/gstglwindow_x11_glx.c
index 7a796a6cf6..af1aaeccb6 100644
--- a/gst-libs/gst/gl/x11/gstglwindow_x11_glx.c
+++ b/gst-libs/gst/gl/x11/gstglwindow_x11_glx.c
@@ -35,6 +35,9 @@
 G_DEFINE_TYPE (GstGLWindowX11GLX, gst_gl_window_x11_glx,
     GST_GL_TYPE_WINDOW_X11);
 
+#define GST_GL_WINDOW_X11_GLX_GET_PRIVATE(o) \
+  (G_TYPE_INSTANCE_GET_PRIVATE((o), GST_GL_TYPE_WINDOW_X11_GLX, GstGLWindowX11GLXPrivate))
+
 static guintptr gst_gl_window_x11_glx_get_gl_context (GstGLWindowX11 *
     window_x11);
 static void gst_gl_window_x11_glx_swap_buffers (GstGLWindowX11 * window_x11);
@@ -49,12 +52,20 @@ GstGLAPI gst_gl_window_x11_glx_get_gl_api (GstGLWindow * window);
 static gpointer gst_gl_window_x11_glx_get_proc_address (GstGLWindow * window,
     const gchar * name);
 
+struct _GstGLWindowX11GLXPrivate
+{
+  int glx_major;
+  int glx_minor;
+};
+
 static void
 gst_gl_window_x11_glx_class_init (GstGLWindowX11GLXClass * klass)
 {
   GstGLWindowClass *window_class = (GstGLWindowClass *) klass;
   GstGLWindowX11Class *window_x11_class = (GstGLWindowX11Class *) klass;
 
+  g_type_class_add_private (klass, sizeof (GstGLWindowX11GLXPrivate));
+
   window_x11_class->get_gl_context =
       GST_DEBUG_FUNCPTR (gst_gl_window_x11_glx_get_gl_context);
   window_x11_class->activate =
@@ -77,6 +88,7 @@ gst_gl_window_x11_glx_class_init (GstGLWindowX11GLXClass * klass)
 static void
 gst_gl_window_x11_glx_init (GstGLWindowX11GLX * window)
 {
+  window->priv = GST_GL_WINDOW_X11_GLX_GET_PRIVATE (window);
 }
 
 /* Must be called in the gl thread */
@@ -130,16 +142,11 @@ static gboolean
 gst_gl_window_x11_glx_choose_format (GstGLWindowX11 * window_x11,
     GError ** error)
 {
+  GstGLWindowX11GLX *window_glx;
   gint error_base;
   gint event_base;
 
-  gint attrib[] = {
-    GLX_RGBA,
-    GLX_RED_SIZE, 1, GLX_GREEN_SIZE, 1, GLX_BLUE_SIZE, 1,
-    GLX_DEPTH_SIZE, 16,
-    GLX_DOUBLEBUFFER,
-    None
-  };
+  window_glx = GST_GL_WINDOW_X11_GLX (window_x11);
 
   if (!glXQueryExtension (window_x11->device, &error_base, &event_base)) {
     g_set_error (error, GST_GL_WINDOW_ERROR,
@@ -147,15 +154,71 @@ gst_gl_window_x11_glx_choose_format (GstGLWindowX11 * window_x11,
     goto failure;
   }
 
-  window_x11->visual_info = glXChooseVisual (window_x11->device,
-      window_x11->screen_num, attrib);
-
-  if (!window_x11->visual_info) {
-    g_set_error (error, GST_GL_WINDOW_ERROR, GST_GL_WINDOW_ERROR_WRONG_CONFIG,
-        "Bad attributes in glXChooseVisual");
+  if (!glXQueryVersion (window_x11->device, &window_glx->priv->glx_major,
+          &window_glx->priv->glx_minor)) {
+    g_set_error (error, GST_GL_WINDOW_ERROR, GST_GL_WINDOW_ERROR_CREATE_CONTEXT,
+        "Failed to query GLX version (glXQueryVersion failed)");
     goto failure;
   }
 
+  GST_INFO ("GLX Version: %d.%d", window_glx->priv->glx_major,
+      window_glx->priv->glx_minor);
+
+  /* legacy case */
+  if (window_glx->priv->glx_major < 1 || (window_glx->priv->glx_major == 1
+          && window_glx->priv->glx_minor < 3)) {
+    gint attribs[] = {
+      GLX_RGBA,
+      GLX_RED_SIZE, 1,
+      GLX_GREEN_SIZE, 1,
+      GLX_BLUE_SIZE, 1,
+      GLX_DEPTH_SIZE, 16,
+      GLX_DOUBLEBUFFER,
+      None
+    };
+
+    window_x11->visual_info = glXChooseVisual (window_x11->device,
+        window_x11->screen_num, attribs);
+
+    if (!window_x11->visual_info) {
+      g_set_error (error, GST_GL_WINDOW_ERROR, GST_GL_WINDOW_ERROR_WRONG_CONFIG,
+          "Bad attributes in glXChooseVisual");
+      goto failure;
+    }
+  } else {
+    gint attribs[] = {
+      GLX_RENDER_TYPE, GLX_RGBA_BIT,
+      GLX_RED_SIZE, 1,
+      GLX_GREEN_SIZE, 1,
+      GLX_BLUE_SIZE, 1,
+      GLX_DEPTH_SIZE, 16,
+      GLX_DOUBLEBUFFER, True,
+      None
+    };
+
+    int fbcount;
+
+    GLXFBConfig *fbconfigs = glXChooseFBConfig (window_x11->device,
+        DefaultScreen (window_x11->device), attribs, &fbcount);
+
+    if (!fbconfigs) {
+      g_set_error (error, GST_GL_WINDOW_ERROR, GST_GL_WINDOW_ERROR_WRONG_CONFIG,
+          "Could not find any FBConfig's to use (check attributes?)");
+      goto failure;
+    }
+
+    window_x11->visual_info = glXGetVisualFromFBConfig (window_x11->device,
+        fbconfigs[0]);
+
+    XFree (fbconfigs);
+
+    if (!window_x11->visual_info) {
+      g_set_error (error, GST_GL_WINDOW_ERROR, GST_GL_WINDOW_ERROR_WRONG_CONFIG,
+          "Bad attributes in FBConfig");
+      goto failure;
+    }
+  }
+
   return TRUE;
 
 failure:
diff --git a/gst-libs/gst/gl/x11/gstglwindow_x11_glx.h b/gst-libs/gst/gl/x11/gstglwindow_x11_glx.h
index 79eaf13a43..4930054a25 100644
--- a/gst-libs/gst/gl/x11/gstglwindow_x11_glx.h
+++ b/gst-libs/gst/gl/x11/gstglwindow_x11_glx.h
@@ -36,13 +36,16 @@ G_BEGIN_DECLS
 
 typedef struct _GstGLWindowX11GLX        GstGLWindowX11GLX;
 typedef struct _GstGLWindowX11GLXClass   GstGLWindowX11GLXClass;
+typedef struct _GstGLWindowX11GLXPrivate GstGLWindowX11GLXPrivate;
 
 struct _GstGLWindowX11GLX {
   /*< private >*/
   GstGLWindowX11 parent;
   
   GLXContext glx_context;
-  
+
+  GstGLWindowX11GLXPrivate *priv;
+
   gpointer _reserved[GST_PADDING];
 };