On Thu, Oct 21, 2004 at 11:49:20PM +0200, Dieter N�tzel wrote: > Am Freitag, 15. Oktober 2004 22:51 schrieb Nicolai Haehnle: > > Hi, > > > > There is disagreement about the meaning of the CLIPSPAN _n parameter in > > CVS. > > > > The drivers I have looked at and drivers/dri/common/spantmp.h treat _n as > > the number of pixels in the span after clipping. > > depthtmp.h and stenciltmp.h treat _n as the end+1 x coordinate of the span. > > > > This inconsistency leads to artifacts when software fallbacks are hit while > > clipping is used, especially with partially obscured clients. The attached > > patch should fix these artifacts by changing depthtmp.h and stenciltmp.h > > appropriately. > > What about this? > > Needed?
AFAICS the code is clearly broken and the fix is correct. The explanation was a bit misleading though as it's the _n1 parameter and not _n that is used incorrectly. -- Ville Syrj�l� [EMAIL PROTECTED] http://www.sci.fi/~syrjala/ ------------------------------------------------------- This SF.net email is sponsored by: IT Product Guide on ITManagersJournal Use IT products in your business? Tell us what you think of them. Give us Your Opinions, Get Free ThinkGeek Gift Certificates! Click to find out more http://productguide.itmanagersjournal.com/guidepromo.tmpl -- _______________________________________________ Dri-devel mailing list [EMAIL PROTECTED] https://lists.sourceforge.net/lists/listinfo/dri-devel
