[ODE] Patch for void dxCylinder::computeAABB()

Geoff Carlton gcarlton at iinet.net.au
Mon Feb 6 16:17:16 MST 2006


I added that a few hours after I received Morgan's email.

Geoff


Bram Stolk wrote:
> Are you sure about this????
> Your unstable tree may be out of date?
> For me, it reads:
>
> void dxCylinder::computeAABB()
> {
>     const dMatrix3& R = final_posr->R;
>     const dVector3& pos = final_posr->pos;
>
>         dReal xrange = dFabs (R[0] * radius) +   dFabs (R[1] * radius) + REAL(0.5)* dFabs (R[2] *
>                 lz);
>         dReal yrange = dFabs (R[4] * radius) +   dFabs (R[5] * radius) + REAL(0.5)* dFabs (R[6] *
>                 lz);
>         dReal zrange = dFabs (R[8] * radius) +   dFabs (R[9] * radius) + REAL(0.5)* dFabs (R[10] *
>                 lz);
>         aabb[0] = pos[0] - xrange;
>         aabb[1] = pos[0] + xrange;
>         aabb[2] = pos[1] - yrange;
>         aabb[3] = pos[1] + yrange;
>         aabb[4] = pos[2] - zrange;
>         aabb[5] = pos[2] + zrange;
> }
>
> So, it already has the final_posr part.
>
>   Bram
>
>
>
> -----Original Message-----
> From: ode-bounces at q12.org on behalf of Morgan McGuire
> Sent: Sun 2/5/2006 05:06
> To: ode at q12.org
> Subject: [ODE] Patch for void dxCylinder::computeAABB()
>  
> The current UNSTABLE CVS dxCylinder is missing "final_posr->" in front of
> several members (see below).
>
> -m
>
> void dxCylinder::computeAABB()
> {
> 	dReal xrange = dFabs (final_posr->R[0] * radius) +	 dFabs (final_posr->R[1] *
> radius) + REAL(0.5)* dFabs (final_posr->R[2] * 
> 		lz);
> 	dReal yrange = dFabs (final_posr->R[4] * radius) +   dFabs (final_posr->R[5] *
> radius) + REAL(0.5)* dFabs (final_posr->R[6] * 
> 		lz);
> 	dReal zrange = dFabs (final_posr->R[8] * radius) +	 dFabs (final_posr->R[9] *
> radius) + REAL(0.5)* dFabs (final_posr->R[10] * 
> 		lz);
> 	aabb[0] = final_posr->pos[0] - xrange;
> 	aabb[1] = final_posr->pos[0] + xrange;
> 	aabb[2] = final_posr->pos[1] - yrange;
> 	aabb[3] = final_posr->pos[1] + yrange;
> 	aabb[4] = final_posr->pos[2] - zrange;
> 	aabb[5] = final_posr->pos[2] + zrange;
> }
> _______________________________________________
> ODE mailing list
> ODE at q12.org
> http://q12.org/mailman/listinfo/ode
>
>
> _______________________________________________
> ODE mailing list
> ODE at q12.org
> http://q12.org/mailman/listinfo/ode
>
>
>   


More information about the ODE mailing list