On Tue, Mar 25, 2003 at 11:26:39PM +0100, Felix Kühling wrote:
> On Tue, 25 Mar 2003 20:02:24 +0000
> Keith Whitwell <[EMAIL PROTECTED]> wrote:
>
> > Gareth Hughes wrote:
> > > Andreas Stenglein wrote:
> > >
> > >>
> > >> Yes, at least the part with GL_TRIANGLE_STRIP.
> > >> In case of "0" you can just return 0, no copying is needed.
> > >>
> > >> case 0: return 0; break;
> > >
> > >
> > > You're going to do that, just in a slightly different manner:
> > >
> > > switch (nr) {
> > > case 0: ovf = 0; break;
> > > case 1: ovf = 1; break;
> > > default: ovf = 2; break;
> > > }
>
> This is the same as
>
> ovf = MIN2( nr, 2 );
>
> Nice to see in how many creative ways this can be obfuscated. ;-) So it
> was just an off-by-one error in the original version, which was
>
> ovf = MIN2( nr-1, 2 );
Hmmm, this looks very much like it could be the reason for the problems I
reported the other day: http://cpbotha.net/thingies/dri_scapula.png
That scapula consists mostly out of triangle strips. :)
--
charl p. botha http://cpbotha.net/ http://visualisation.tudelft.nl/
-------------------------------------------------------
This SF.net email is sponsored by:
The Definitive IT and Networking Event. Be There!
NetWorld+Interop Las Vegas 2003 -- Register today!
http://ads.sourceforge.net/cgi-bin/redirect.pl?keyn0001en
_______________________________________________
Dri-devel mailing list
[EMAIL PROTECTED]
https://lists.sourceforge.net/lists/listinfo/dri-devel
