http://bugs.freedesktop.org/show_bug.cgi?id=9962





------- Comment #6 from [EMAIL PROTECTED]  2007-08-03 09:08 PST -------

> 2. GL_TRIANGLE_STRIP is a special case when spliting
> for example: 
> the vertices are v0, v1, v2, v3, v4, v5, v6 ...
> so triangles should be(v0, v1, v2) (v2, v1, v3) (v2, v3, v4), (v4, v3, v5)...
> if the vertices was splited between v4 and v5, part 2 would be (v3, v4), v5,
> v6... so the first triangle of part 2 is (v3, v4, v5), note the clock order
> differ from (v4, v3, v5). It is severe if GL_CULL_FACE is enabled. 
> This patch ensure part1 and part2 have the same clock order.


Looks good, thanks for catching this.

Your patch includes slightly more logic than I think is necessary, would
something like the following work as well?

   if (mode == GL_TRIANGLE_STRIP &&
       (copy->dstelt_nr & 1))
      return GL_FALSE;


-- 
Configure bugmail: http://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.

-------------------------------------------------------------------------
This SF.net email is sponsored by: Splunk Inc.
Still grepping through log files to find problems?  Stop.
Now Search log events and configuration files using AJAX and a browser.
Download your FREE copy of Splunk now >>  http://get.splunk.com/
--
_______________________________________________
Dri-devel mailing list
Dri-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/dri-devel

Reply via email to