registry team mailing list archive

Thread
Date

[Bug 550218]

To: registry@xxxxxxxxxxxxxxxxxxx
From: Peng-li <550218@xxxxxxxxxxxxxxxxxx>
Date: Tue, 14 Sep 2010 01:02:01 -0000
Reply-to: Bug 550218 <550218@xxxxxxxxxxxxxxxxxx>
Sender: bounces@xxxxxxxxxxxxx

Kristian:

I have a question about the patch "DRI2: Track DRI2 drawables as
resources, not privates", it calls AddResource() in DRI2CreateDrawable()
to add DRI2 drawable as resources, but where we free them, should we
call FreeResourceByType() to free the DRI2 drawable in
ProcDRI2DestroyDrawable() ?

Also in my test in MeeGo. I find that in ProcDRI2CreateDrawable(),
sometimes the value of "pDrawable->id" is 0, which is different with
stuff->drawable. that would cause some problem. because we will get an
old DRI2 drawable from DRI2GetDrawable() that attached to the zero value
of "pDrawable->id". I think we need to add some check to make sure
"pDrawable->id" is synced with "stuff->drawable".

I attached a draft patch that addressing above two points, please
review.

-- 
xserver crashes when closing application using clutter
https://bugs.launchpad.net/bugs/550218
You received this bug notification because you are a member of Registry
Administrators, which is the registrant for Debian.