On Sun, 2010-05-02 at 16:48 -0700, Keith Packard wrote:
> On Sun, 2 May 2010 17:26:58 -0400, Kristian Høgsberg <[email protected]> 
> wrote:
> > I guess they don't do it either... bad assumption on my part.  But it
> > *is* a bug, right?  I can fix those too and resend, but I'm just still
> > surprised that this bug was in there all the time and not sure if
> > instead I'm missing something subtle in the resource code.
> 
> FreeResourceByType was added for X11R4 with this bug in place, long
> after the authors of the resource code were gone, so I suspect the whole
> 'elements' invariant was lost. I'd love to know why it was added then;
> the server didn't use it at all then, and the core code still does not.

Incorrect:

atropine:/tmp/r4/mit% grep FreeResourceByType **/*.[ch]
extensions/server/multibuf.c:    FreeResourceByType (pWin->drawable.id, 
MultibuffersResType, FALSE);
server/dix/resource.c:FreeResourceByType(id, type, skipFree)

Which makes sense; destroying a window should destroy all the image
buffers bound to it.

- ajax

Attachment: signature.asc
Description: This is a digitally signed message part

_______________________________________________
[email protected]: X.Org development
Archives: http://lists.x.org/archives/xorg-devel
Info: http://lists.x.org/mailman/listinfo/xorg-devel

Reply via email to