Re: [ggt-devel] Slight optimization to isInVolume()

[Kevin M. <ke...@su...>]

Yep.  That's a good one.

---
kevin meinert | http://www.subatomiclabs.com


On Fri, Oct 26, 2012 at 10:00 AM, Paul Martz <pm...@sk...> wrote:

> Seems like the VecOps.h length() function will almost always incur a
> sqrt(). I think we can avoid calling length() from the
> isInVolume(Sphere,Point) test with the attached patch, which makes
> isInVolume() compare the squared length to the squared radius. This trades
> a sqrt() for a single multiply operation.
>
> Submitting this as a separate patch as it is completely orthogonal to my
> other work.
>    -Paul
>
>
>
>
>
> ------------------------------------------------------------------------------
> Everyone hates slow websites. So do we.
> Make your web apps faster with AppDynamics
> Download AppDynamics Lite for free today:
> http://p.sf.net/sfu/appdyn_sfd2d_oct
> _______________________________________________
> ggt-devel mailing list
> ggt...@li...
> https://lists.sourceforge.net/lists/listinfo/ggt-devel
>
>