Added Pair.update optimisations and removed the Contact class.

Author: photonstorm

v3/src/physics/matter-js/CustomMain.js

@@ -4,7 +4,6 @@ Matter.Body = require('./lib/body/Body');
 Matter.Composite = require('./lib/body/Composite');
 Matter.World = require('./lib/body/World');
 
-Matter.Contact = require('./lib/collision/Contact');
 Matter.Detector = require('./lib/collision/Detector');
 Matter.Grid = require('./lib/collision/Grid');
 Matter.Pairs = require('./lib/collision/Pairs');

v3/src/physics/matter-js/lib/collision/Contact.js

@@ -8,8 +8,6 @@ var Pair = {};
 
 module.exports = Pair;
 
-var Contact = require('./Contact');
-
 (function() {
 
     /**
@@ -21,27 +19,26 @@ var Contact = require('./Contact');
      */
     Pair.create = function(collision, timestamp) {
         var bodyA = collision.bodyA,
-            bodyB = collision.bodyB,
-            parentA = collision.parentA,
-            parentB = collision.parentB;
+            bodyB = collision.bodyB;
 
         var pair = {
             id: Pair.id(bodyA, bodyB),
             bodyA: bodyA,
             bodyB: bodyB,
-            contacts: {},
             activeContacts: [],
             separation: 0,
             isActive: true,
             confirmedActive: true,
             isSensor: bodyA.isSensor || bodyB.isSensor,
             timeCreated: timestamp,
             timeUpdated: timestamp,
-            inverseMass: parentA.inverseMass + parentB.inverseMass,
-            friction: Math.min(parentA.friction, parentB.friction),
-            frictionStatic: Math.max(parentA.frictionStatic, parentB.frictionStatic),
-            restitution: Math.max(parentA.restitution, parentB.restitution),
-            slop: Math.max(parentA.slop, parentB.slop)
+
+            collision: null,
+            inverseMass: 0,
+            friction: 0,
+            frictionStatic: 0,
+            restitution: 0,
+            slop: 0
         };
 
         Pair.update(pair, collision, timestamp);
@@ -57,31 +54,41 @@ var Contact = require('./Contact');
      * @param {number} timestamp
      */
     Pair.update = function(pair, collision, timestamp) {
-        var contacts = pair.contacts,
-            supports = collision.supports,
-            activeContacts = pair.activeContacts,
-            parentA = collision.parentA,
-            parentB = collision.parentB;
+        // var contacts = pair.contacts,
+        //     supports = collision.supports,
+        //     activeContacts = pair.activeContacts,
+        //     parentA = collision.parentA,
+        //     parentB = collision.parentB;
 
         pair.collision = collision;
-        pair.inverseMass = parentA.inverseMass + parentB.inverseMass;
-        pair.friction = Math.min(parentA.friction, parentB.friction);
-        pair.frictionStatic = Math.max(parentA.frictionStatic, parentB.frictionStatic);
-        pair.restitution = Math.max(parentA.restitution, parentB.restitution);
-        pair.slop = Math.max(parentA.slop, parentB.slop);
-        activeContacts.length = 0;
+        // pair.inverseMass = parentA.inverseMass + parentB.inverseMass;
+        // pair.friction = Math.min(parentA.friction, parentB.friction);
+        // pair.frictionStatic = Math.max(parentA.frictionStatic, parentB.frictionStatic);
+        // pair.restitution = Math.max(parentA.restitution, parentB.restitution);
+        // pair.slop = Math.max(parentA.slop, parentB.slop);
+        // activeContacts.length = 0;
 
         if (collision.collided) {
+
+            var supports = collision.supports,
+                activeContacts = pair.activeContacts,
+                parentA = collision.parentA,
+                parentB = collision.parentB;
+
+            pair.inverseMass = parentA.inverseMass + parentB.inverseMass;
+            pair.friction = Math.min(parentA.friction, parentB.friction);
+            pair.frictionStatic = Math.max(parentA.frictionStatic, parentB.frictionStatic);
+            pair.restitution = Math.max(parentA.restitution, parentB.restitution);
+            pair.slop = Math.max(parentA.slop, parentB.slop);
+
             for (var i = 0; i < supports.length; i++) {
-                var support = supports[i],
-                    contactId = Contact.id(support),
-                    contact = contacts[contactId];
-
-                if (contact) {
-                    activeContacts.push(contact);
-                } else {
-                    activeContacts.push(contacts[contactId] = Contact.create(support));
-                }
+                activeContacts[i] = supports[i].contact;
+            }
+
+            var supportCount = supports.length;
+
+            if (supportCount < activeContacts.length) {
+                activeContacts.length = supportCount;
             }
 
             pair.separation = collision.depth;

v3/src/physics/matter-js/lib/collision/Pair.js

@@ -44,9 +44,16 @@ var Common = require('../core/Common');
                     y: point.y,
                     index: i,
                     body: body,
-                    isInternal: false
+                    isInternal: false,
+                    contact: null
                 };
 
+            vertex.contact = {
+                vertex: vertex,
+                normalImpulse: 0,
+                tangentImpulse: 0
+            };
+
             vertices.push(vertex);
         }