search

favreau / bullet

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

resize(0) instead of clear() #459

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago 
Hi Bullet Developers, 

It seems that following the advice in alignedObjectArray and changing 
btCompoundCollisionAlgorithm.cpp, line 237 from 

 manifoldArray.clear();

to 

 manifoldArray.resize(0);

can have quite a performance impact on scenes with lots of compound
objects (we see about 25% speedup). 

Might be worthwhile to change.. 

Thanks,

Peter Messmer

Original issue reported on code.google.com by peter.me...@gmail.com on 7 Dec 2010 at 6:57

GoogleCodeExporter commented 9 years ago 

Good find, we'll have a look at it and apply it soon.

Thanks!

Original comment by erwin.coumans on 7 Dec 2010 at 7:12

GoogleCodeExporter commented 9 years ago 

Patch was applied in trunk, see 
http://code.google.com/p/bullet/source/detail?r=2245

Thanks for the feedback!

Original comment by erwin.coumans on 13 Dec 2010 at 11:37