Move DRI drawable creation into dri_glx.c.

diff --git a/src/glx/x11/dri_glx.c b/src/glx/x11/dri_glx.c
index c96356fd634bbf00df2b9dd470b5732d3f372e67..2e883c62cb79a6c463b9ae22854a950c2ad4b9c7 100644 (file)
--- a/src/glx/x11/dri_glx.c
+++ b/src/glx/x11/dri_glx.c
@@ -413,8 +413,8 @@ __glXDRIGetDrawableInfo(__DRIdrawable *drawable,
                        int *backX, int *backY,
                        int *numBackClipRects, drm_clip_rect_t **pBackClipRects)
 {
-    __GLXdrawable *glxDraw =
-       containerOf(drawable, __GLXdrawable, driDrawable);
+    __GLXDRIdrawable *glxDraw =
+       containerOf(drawable, __GLXDRIdrawable, driDrawable);
     __GLXscreenConfigs *psc = glxDraw->psc;
     Display *dpy = psc->dpy;
 
@@ -693,6 +693,50 @@ static void driCreateContext(__GLXscreenConfigs *psc,
     }
 }
 
+
+static __GLXDRIdrawable *driCreateDrawable(__GLXscreenConfigs *psc,
+                                          GLXDrawable drawable,
+                                          GLXContext gc)
+{
+    __GLXDRIdrawable *pdraw;
+    drm_drawable_t hwDrawable;
+    void *empty_attribute_list = NULL;
+
+    pdraw = Xmalloc(sizeof(*pdraw));
+    if (!pdraw)
+       return NULL;
+
+    pdraw->drawable = drawable;
+    pdraw->psc = psc;
+
+    if (!XF86DRICreateDrawable(psc->dpy, psc->scr, drawable, &hwDrawable))
+       return NULL;
+
+    /* Create a new drawable */
+    pdraw->driDrawable.private =
+       (*psc->__driScreen.createNewDrawable)(&psc->__driScreen,
+                                             gc->mode,
+                                             &pdraw->driDrawable,
+                                             hwDrawable,
+                                             GLX_WINDOW_BIT,
+                                             empty_attribute_list);
+
+    if (!pdraw->driDrawable.private) {
+       XF86DRIDestroyDrawable(psc->dpy, psc->scr, drawable);
+       Xfree(pdraw);
+       return NULL;
+    }
+
+    if (__glxHashInsert(psc->drawHash, drawable, pdraw)) {
+       (*pdraw->driDrawable.destroyDrawable)(&pdraw->driDrawable);
+       XF86DRIDestroyDrawable(psc->dpy, psc->scr, drawable);
+       Xfree(pdraw);
+       return NULL;
+    }
+
+    return pdraw;
+}
+
 static void driDestroyScreen(__GLXscreenConfigs *psc)
 {
     /* Free the direct rendering per screen data */
@@ -737,6 +781,7 @@ static __GLXDRIscreen *driCreateScreen(__GLXscreenConfigs *psc, int screen,
 
     psp->destroyScreen = driDestroyScreen;
     psp->createContext = driCreateContext;
+    psp->createDrawable = driCreateDrawable;
 
     return psp;
 }

diff --git a/src/glx/x11/glxclient.h b/src/glx/x11/glxclient.h
index b8ba49673872b77926990efe5516222358b4b8c6..d99918a185e959fee332e49f17464d2865242911 100644 (file)
--- a/src/glx/x11/glxclient.h
+++ b/src/glx/x11/glxclient.h
@@ -94,6 +94,7 @@ typedef struct _glapi_table __GLapi;
  */
 typedef struct __GLXDRIdisplayRec __GLXDRIdisplay;
 typedef struct __GLXDRIscreenRec __GLXDRIscreen;
+typedef struct __GLXDRIdrawableRec __GLXDRIdrawable;
 
 struct __GLXDRIdisplayRec {
     /**
@@ -112,6 +113,16 @@ struct __GLXDRIscreenRec {
     void (*createContext)(__GLXscreenConfigs *psc,
                          const __GLcontextModes *mode,
                          GLXContext gc, GLXContext shareList, int renderType);
+
+    __GLXDRIdrawable *(*createDrawable)(__GLXscreenConfigs *psc,
+                                       GLXDrawable drawable,
+                                       GLXContext gc);
+};
+
+struct __GLXDRIdrawableRec {
+    XID drawable;
+    __GLXscreenConfigs *psc;
+    __DRIdrawable driDrawable;
 };
 
 /*
@@ -541,17 +552,6 @@ struct __GLXdisplayPrivateRec {
 #endif
 };
 
-#ifdef GLX_DIRECT_RENDERING
-
-struct __GLXdrawableRec {
-    XID drawable;
-    __GLXscreenConfigs *psc;
-    __DRIdrawable driDrawable;
-};
-
-#endif
-
-
 
 void __glXFreeContext(__GLXcontext*);
 

diff --git a/src/glx/x11/glxcmds.c b/src/glx/x11/glxcmds.c
index 15c9d19c97d25c171d370a22082c7831d2903673..6b34acfcf3492e9111ec62e3419955b384854086 100644 (file)
--- a/src/glx/x11/glxcmds.c
+++ b/src/glx/x11/glxcmds.c
@@ -83,7 +83,7 @@ static int windowExistsErrorHandler(Display *dpy, XErrorEvent *xerr)
 static void GarbageCollectDRIDrawables(Display *dpy, __GLXscreenConfigs *sc)
 {
     XID draw;
-    __GLXdrawable *pdraw;
+    __GLXDRIdrawable *pdraw;
     XWindowAttributes xwa;
     int (*oldXErrorHandler)(Display *, XErrorEvent *);
 
@@ -124,7 +124,7 @@ static __DRIdrawable *
 GetDRIDrawable( Display *dpy, GLXDrawable drawable, int * const scrn_num )
 {
     __GLXdisplayPrivate * const priv = __glXInitialize(dpy);
-    __GLXdrawable * const pdraw;
+    __GLXDRIdrawable * const pdraw;
     const unsigned  screen_count = ScreenCount(dpy);
     unsigned   i;
     __GLXscreenConfigs *sc;
@@ -2143,9 +2143,9 @@ __driGetMscRateOML(__DRIdrawable *draw, int32_t *numerator, int32_t *denominator
     XF86VidModeModeLine   mode_line;
     int   dot_clock;
     int   i;
-    __GLXdrawable *glxDraw;
+    __GLXDRIdrawable *glxDraw;
 
-    glxDraw = containerOf(draw, __GLXdrawable, driDrawable);
+    glxDraw = containerOf(draw, __GLXDRIdrawable, driDrawable);
     psc = glxDraw->psc;
     if (XF86VidModeQueryVersion(psc->dpy, &i, &i) &&
        XF86VidModeGetModeLine(psc->dpy, psc->scr, &dot_clock, &mode_line) ) {

diff --git a/src/glx/x11/glxext.c b/src/glx/x11/glxext.c
index 372dbfca657990e48176712a0bc7d6b74bdfef26..2f3261801a6f70ce42001dd90f3635d4b47d0a75 100644 (file)
--- a/src/glx/x11/glxext.c
+++ b/src/glx/x11/glxext.c
@@ -1178,53 +1178,20 @@ static Bool SendMakeCurrentRequest(Display *dpy, CARD8 opcode,
 
 #ifdef GLX_DIRECT_RENDERING
 static __DRIdrawable *
-FetchDRIDrawable( Display *dpy, GLXDrawable drawable, GLXContext gc)
+FetchDRIDrawable(Display *dpy, GLXDrawable drawable, GLXContext gc)
 {
     __GLXdisplayPrivate * const priv = __glXInitialize(dpy);
-    __GLXdrawable *pdraw;
-    __GLXscreenConfigs *sc;
-    drm_drawable_t hwDrawable;
-    void *empty_attribute_list = NULL;
+    __GLXDRIdrawable *pdraw;
+    __GLXscreenConfigs *psc;
 
     if (priv == NULL || priv->driDisplay == NULL)
        return NULL;
     
-    sc = &priv->screenConfigs[gc->screen];
-    if (__glxHashLookup(sc->drawHash, drawable, (void *) &pdraw) == 0)
+    psc = &priv->screenConfigs[gc->screen];
+    if (__glxHashLookup(psc->drawHash, drawable, (void *) &pdraw) == 0)
        return &pdraw->driDrawable;
 
-    /* Allocate a new drawable */
-    pdraw = Xmalloc(sizeof(*pdraw));
-    if (!pdraw)
-       return NULL;
-
-    pdraw->drawable = drawable;
-    pdraw->psc = sc;
-
-    if (!XF86DRICreateDrawable(dpy, sc->scr, drawable, &hwDrawable))
-       return NULL;
-
-    /* Create a new drawable */
-    pdraw->driDrawable.private =
-       (*sc->__driScreen.createNewDrawable)(&sc->__driScreen,
-                                            gc->mode,
-                                            &pdraw->driDrawable,
-                                            hwDrawable,
-                                            GLX_WINDOW_BIT,
-                                            empty_attribute_list);
-
-    if (!pdraw->driDrawable.private) {
-       XF86DRIDestroyDrawable(dpy, sc->scr, drawable);
-       Xfree(pdraw);
-       return NULL;
-    }
-
-    if (__glxHashInsert(sc->drawHash, drawable, pdraw)) {
-       (*pdraw->driDrawable.destroyDrawable)(&pdraw->driDrawable);
-       XF86DRIDestroyDrawable(dpy, sc->scr, drawable);
-       Xfree(pdraw);
-       return NULL;
-    }
+    pdraw = psc->driScreen->createDrawable(psc, drawable, gc);
 
     return &pdraw->driDrawable;
 }