ammo.js icon indicating copy to clipboard operation
ammo.js copied to clipboard

btCollisionWorld constructors not defined

Open stevenvergenz opened this issue 7 years ago • 3 comments

I noticed that btCollisionWorld doesn't have any constructors defined in the IDL. This class is usually just a super class, but it's useful on its own too if you only need collision detection and no simulation, and this use case is currently broken. In fact, the only classes with constructors in that inheritance tree is btDiscreteDynamicsWorld and btSoftRigidDynamicsWorld.

Could you add at least the basic constructor for this class? I can file a PR if you'd like, but I haven't set up the build environment yet, and it looks like it'd take some time. This is probably a one-line change, seems like overkill.

stevenvergenz avatar Dec 21 '17 00:12 stevenvergenz

While you're at it, btCollisionObject is also missing a constructor, which is used for the same use case.

stevenvergenz avatar Dec 21 '17 00:12 stevenvergenz

#182 added some of this I think.

If you open a PR with what you need, I can test it, so you don't need a full build environment.

kripken avatar Dec 21 '17 18:12 kripken

I'm getting "Uncaught cannot construct a btCollisionWorld, no constructor in IDL" and happens the same with btCollisionObject. Im trying to do collision detection alone with no physics simulation

mariocesarc avatar Oct 19 '18 15:10 mariocesarc