schteppe / p2.js

JavaScript 2D physics library
Other
2.64k stars 330 forks source link

world.removeBody() does not remove pairs from world.disabledBodyCollisionPairs #193

Closed psalaets closed 8 years ago

psalaets commented 8 years ago

I think this is a memory leak.

This test case fails (running against master branch):

var a = new Body(),
    b = new Body(),
    world = new World();
world.addBody(a);
world.addBody(b);
world.disableBodyCollision(a, a);
world.disableBodyCollision(b, b);

world.removeBody(a);

test.deepEqual(world.disabledBodyCollisionPairs, [b, b]);
test.done();