Please do not reply to this email: if you want to comment on the bug, go to    
       
the URL shown below and enter yourcomments there.     
   
https://bugs.freedesktop.org/show_bug.cgi?id=5714          
     




------- Additional Comments From [EMAIL PROTECTED]  2007-01-04 15:14 -------
(In reply to comment #20)
> I'd like to recommend a couple additional changes.
> 
> - Remove the parameters to unbindContext that you noted were unnecessary.
> 
> - Replace the dpy/screen tuple that is passed to many functions with a
> __DRIscreen pointer.  It appears that the dpy is passed into the driver just 
> so
> that the driver can pass it back to the loader.  Instead store the dpy/screen 
> in
> the loader private __DRIscreen data (screenConfigs).  This would also 
> eliminate
> all calls to getScreen.
> 
> - Remove getScreen.

These changes all make a lot of sense, I've wanted to do something similar in a
follow up patch.  In fact, we shouldn't publish a new DRI interface version
without these changes.  It shouldn't be a lot of work and I may be able to get a
patch done, but I'm temporarily not working on this stuff so I can't guarantee
anything.          
     
     
--           
Configure bugmail: https://bugs.freedesktop.org/userprefs.cgi?tab=email         
     
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.

-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys - and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV
--
_______________________________________________
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel

Reply via email to