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

[Bram Stolk]

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