Possible typo in btDbvt.cpp getmaxdepth()

[jhurliman]

btDbvt.cpp lines 63 and 64 in getmaxdepth() are identical in my checkout of the code.

Code: Select all

static void						getmaxdepth(const btDbvtNode* node,int depth,int& maxdepth)
{
	if(node->isinternal())
	{
		getmaxdepth(node->childs[0],depth+1,maxdepth);
		getmaxdepth(node->childs[0],depth+1,maxdepth);
	} else maxdepth=btMax(maxdepth,depth);
}

That should be node->childs[1] on the second line, correct?

[Erwin Coumans]

This is a bug indeed, fixed now in latest trunk: http://code.google.com/p/bullet/source/detail?r=2076

It seems 'maxdepth' isn't use in the code, is it?

Thanks,
Erwin

[jhurliman]

No I don't think this feature is used in Bullet. I only noticed it when I was implementing my own dynamic AABB tree (in C#) based on the Bullet implementation.