When a drawable is unbound from one direct rendering context and then bound

to another, we do not want to destroy the drawable data structure each time
only to recreate it immediatly afterwards when binding to the next context.
This also causes conflicts with caching of the drawable stamp.

When GLX 1.3 is integrated, the create and destroy drawable functions will
have user level counterparts and the memory will be able to be recovered.
Until that time, we will not destroy the drawables when their refcounts go
to zero.

1 files changed, 15 insertions, 0 deletions

diff --git a/xc/lib/GL/dri/dri_mesa.c b/xc/lib/GL/dri/dri_mesa.c
index ff1ddd4b6..f5fd3fa61 100644
--- a/xc/lib/GL/dri/dri_mesa.c
+++ b/xc/lib/GL/dri/dri_mesa.c
@@ -153,10 +153,25 @@ static Bool driMesaUnbindContext(Display *dpy, int scrn,
 	/* ERROR!!! */
 	return GL_FALSE;
     } else if (--pdp->refcount == 0) {
+#if 0
+	/*
+	** NOT_DONE: When a drawable is unbound from one direct
+	** rendering context and then bound to another, we do not want
+	** to destroy the drawable data structure each time only to
+	** recreate it immediatly afterwards when binding to the next
+	** context.  This also causes conflicts with caching of the
+	** drawable stamp.
+	**
+	** When GLX 1.3 is integrated, the create and destroy drawable
+	** functions will have user level counterparts and the memory
+	** will be able to be recovered.
+	*/
+
 	/* Delete drawable if no longer referenced by any contexts */
 	(*pdraw->destroyDrawable)(dpy, pdraw->private);
 	__driMesaRemoveDrawable(pdraw);
 	Xfree(pdraw);
+#endif
     }
 
     psp = pdp->driScreenPriv;