|
From: Roger H. <rog...@mi...> - 2004-07-22 19:39:54
|
> > That's normally a good trick to use (compare squares rather than actual > lengths), but this code is only hit in a debug build anyway - so it's > probably clearer when you're stepping through to just get the actual > length. Marginally, but I do a lot of testing with the debugging version, and I think sqrt is quite a slow function even on a G4 or G5. > > >> By the way, I think the name kQ3RealZero is a bit misleading. Real >> zero is zero, kQ3RealZero is just a small number. How about >> kQ3AlmostZero? > > Yes, I've always thought kQ3RealZero was an odd name for something that > isn't really zero. :-) > > Perhaps kQ3Epsilon, since that's really what it's used for (and maps to > FLT_EPSILON if that's available). > Yes that would be better. Is it public ? Roger. |