Merge pull request #183 from alexandernanberg/fix-free

fix: don't panic if calling .free() multiple times

Author: sebcrozet

src.ts/dynamics/ccd_solver.ts

@@ -13,7 +13,9 @@ export class CCDSolver {
      * Release the WASM memory occupied by this narrow-phase.
      */
     public free() {
-        this.raw.free();
+        if (!!this.raw) {
+            this.raw.free();
+        }
         this.raw = undefined;
     }
 

src.ts/dynamics/impulse_joint_set.ts

@@ -31,9 +31,14 @@ export class ImpulseJointSet {
      * Release the WASM memory occupied by this joint set.
      */
     public free() {
-        this.raw.free();
+        if (!!this.raw) {
+            this.raw.free();
+        }
         this.raw = undefined;
-        this.map.clear();
+
+        if (!!this.map) {
+            this.map.clear();
+        }
         this.map = undefined;
     }
 

src.ts/dynamics/integration_parameters.ts

@@ -11,7 +11,9 @@ export class IntegrationParameters {
      * Free the WASM memory used by these integration parameters.
      */
     public free() {
-        this.raw.free();
+        if (!!this.raw) {
+            this.raw.free();
+        }
         this.raw = undefined;
     }
 

src.ts/dynamics/island_manager.ts

@@ -14,7 +14,9 @@ export class IslandManager {
      * Release the WASM memory occupied by this narrow-phase.
      */
     public free() {
-        this.raw.free();
+        if (!!this.raw) {
+            this.raw.free();
+        }
         this.raw = undefined;
     }
 

src.ts/dynamics/multibody_joint_set.ts

@@ -30,9 +30,14 @@ export class MultibodyJointSet {
      * Release the WASM memory occupied by this joint set.
      */
     public free() {
-        this.raw.free();
+        if (!!this.raw) {
+            this.raw.free();
+        }
         this.raw = undefined;
-        this.map.clear();
+
+        if (!!this.map) {
+            this.map.clear();
+        }
         this.map = undefined;
     }
 

src.ts/dynamics/rigid_body_set.ts

@@ -21,9 +21,14 @@ export class RigidBodySet {
      * Release the WASM memory occupied by this rigid-body set.
      */
     public free() {
-        this.raw.free();
+        if (!!this.raw) {
+            this.raw.free();
+        }
         this.raw = undefined;
-        this.map.clear();
+
+        if (!!this.map) {
+            this.map.clear();
+        }
         this.map = undefined;
     }
 

src.ts/geometry/broad_phase.ts

@@ -13,7 +13,9 @@ export class BroadPhase {
      * Release the WASM memory occupied by this broad-phase.
      */
     public free() {
-        this.raw.free();
+        if (!!this.raw) {
+            this.raw.free();
+        }
         this.raw = undefined;
     }
 

src.ts/geometry/collider_set.ts

@@ -19,9 +19,14 @@ export class ColliderSet {
      * Release the WASM memory occupied by this collider set.
      */
     public free() {
-        this.raw.free();
+        if (!!this.raw) {
+            this.raw.free();
+        }
         this.raw = undefined;
-        this.map.clear();
+
+        if (!!this.map) {
+            this.map.clear();
+        }
         this.map = undefined;
     }
 

src.ts/geometry/narrow_phase.ts

@@ -16,7 +16,9 @@ export class NarrowPhase {
      * Release the WASM memory occupied by this narrow-phase.
      */
     public free() {
-        this.raw.free();
+        if (!!this.raw) {
+            this.raw.free();
+        }
         this.raw = undefined;
     }
 
@@ -101,7 +103,9 @@ export class TempContactManifold {
     raw: RawContactManifold;
 
     public free() {
-        this.raw.free();
+        if (!!this.raw) {
+            this.raw.free();
+        }
         this.raw = undefined;
     }
 

src.ts/pipeline/debug_render_pipeline.ts

@@ -48,7 +48,9 @@ export class DebugRenderPipeline {
      * Release the WASM memory occupied by this serialization pipeline.
      */
     free() {
-        this.raw.free();
+        if (!!this.raw) {
+            this.raw.free();
+        }
         this.raw = undefined;
         this.vertices = undefined;
         this.colors = undefined;