Limit the accumulator and allow dt of 0, from schteppe#392

pmndrs · Jun 27, 2020 · 2970232 · 2970232
1 parent 157d9b3
commit 2970232
Showing 1 changed file with 5 additions and 3 deletions.
diff --git a/src/world/World.ts b/src/world/World.ts
@@ -385,8 +385,8 @@ export class World extends EventTarget {
    *
    * @see http://bulletphysics.org/mediawiki-1.5.8/index.php/Stepping_The_World
    */
-  step(dt: number, timeSinceLastCalled = 0, maxSubSteps = 10): void {
-    if (timeSinceLastCalled === 0) {
+  step(dt: number, timeSinceLastCalled?: number, maxSubSteps = 10): void {
+    if (timeSinceLastCalled === undefined) {
       // Fixed, simple stepping
 
       this.internalStep(dt)
@@ -410,7 +410,9 @@ export class World extends EventTarget {
         }
       }
 
-      const t = (this.accumulator % dt) / dt
+      this.accumulator = this.accumulator % dt
+
+      const t = this.accumulator / dt
       for (let j = 0; j !== this.bodies.length; j++) {
         const b = this.bodies[j]
         b.previousPosition.lerp(b.position, t, b.interpolatedPosition)