Comment 35 for bug 550218

Revision history for this message
In , Jesse Barnes (jbarnes-virtuousgeek) wrote :

(In reply to comment #26)
> (In reply to comment #25)
> > Such as? Do you have ideas for an alternate approach? Or do you like
> > Kristian's proposed patch better?
>
> I'll leave the argument to Kristian, as he was previously arguing against
> patches of mine using ID lookup, but I thought the alternative approach should
> be clear by now: Clean up all references to the DrawableRec of a window being
> destroyed.

Ok I'll chat with Kristian. The main problem with the cleanup approach is that it's a bit more invasive than the small patch here, and seems to violate some of the existing GLX layering; I was hoping you had something clever to make the impact small. I agree that we'll add some lookup overhead, I'm just not sure how much impact that will have.