|
From: Allen A. <ak...@po...> - 2001-02-02 18:21:54
|
On Sat, Feb 03, 2001 at 02:19:03AM +1100, Gareth Hughes wrote: | So I finally sat down and cleaned up my scissor test a bit. I've just | added it to the repository. Comments welcome, but I'll probably ignore | ones calling me a C hacker (yes, been doing lots of that lately...). Looks pretty good to me. Only three things come to mind: compareOne() still needs to be implemented; testing floating-point color values for exact equality (rather than for 0.5LSB tolerance or something like it) might be risky; and you might want to use GLUtils::useScreenCoords() rather than setting up your own projection and viewport parameters (so that the test checks the setup people are most likely to use, because it's recommended in the Red Book). Nice to have the test, though; scissoring is one of those things that are often overlooked. Allen |