Chinamming / bullet

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

USE_INTERNAL_EDGE_UTILITY define in btWorldImporter no longer compiles #667

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
At some point the old btBulletWorldImporter class code was split into 2 classes 
(most of it moved into btWorldImporter) and apparently the 
USE_INTERNAL_EDGE_UTILITY define to handle treating the normals on internal 
edges of polygons properly wasn't tested. Several problems prevent it from 
compiling in 2.81. Such as the define being required in the header (since its 
checked in 2 .cpps now), the location of btAdjustInternalEdgeContactsCallback 
and the change from passing btCollisionObject pointers to 
btCollisionObjectWrappers to that function. I'm submitting a patch that address 
all of these issues (and fixes a warning). Tested on VS 2008, Win7. I've tested 
cases in our game where internal edges were being detected improperly before 
and they are fine now with this code in place.

Original issue reported on code.google.com by cont...@vicariousentertainment.com on 1 Nov 2012 at 11:55

Attachments:

GoogleCodeExporter commented 9 years ago
See https://github.com/bulletphysics/bullet3/issues/113

Original comment by erwin.coumans on 30 Mar 2014 at 7:21