czaloj / bullet

Automatically exported from code.google.com/p/bullet
0 stars 0 forks source link

Add a CUSTOM_POLYHEDRAL_SHAPE_TYPE #125

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
BroadphaseNativeTypes in btBroadphaseProxy contains a list of shape types. If 
one wants to 
implement a custom polyhedral shape class, one has to use one of the predefined 
polyhedral shape 
types. As all of them have a special meaning (box, triangle, ...), which does 
not fit to the custom 
shape class, one may run into trouble. Currently, I use 
TETRAHEDRAL_SHAPE_PROXYTYPE for 
custom polyhedral shape classes as there is not yet a special handling for this 
shape type in Bullet. 
But a better solution would be to include something like a 
CUSTOM_POLYHEDRAL_SHAPE_TYPE.

The same holds for implicit convex and concave custom shapes. For implicit 
convex shapes, there 
already exists CONVEX_SHAPE_PROXYTYPE which seems to be not used by Bullet, but 
a naming like 
CUSTOM_CONVEX_SHAPE_PROXYTYPE is more explicit.

Original issue reported on code.google.com by ol...@arcor.de on 30 Oct 2008 at 9:38

GoogleCodeExporter commented 9 years ago
Again a very good suggestion, it has been applied in the trunk and will be 
available 
in upcoming Bullet 2.73
http://code.google.com/p/bullet/source/detail?r=1449

Original comment by erwin.coumans on 4 Nov 2008 at 9:43

GoogleCodeExporter commented 9 years ago
Thanks for applying this and the other patches! I noticed some inconsistency of 
my own suggestion: it should 
read CUSTOM_POLYHEDRAL_SHAPE_PROXYTYPE instead of CUSTOM_POLYHEDRAL_SHAPE_TYPE 
to be consistent 
with the other shape types. Sorry for that!

Original comment by ol...@arcor.de on 4 Nov 2008 at 11:09