Skip to content

Commit

Permalink
Nulling instead of deleting in dispose().
Browse files Browse the repository at this point in the history
Still waiting for an "official" recommendation from a VM guy though.
  • Loading branch information
mrdoob committed Dec 23, 2012
1 parent 81dcf8b commit 2466896
Show file tree
Hide file tree
Showing 4 changed files with 4 additions and 4 deletions.
2 changes: 1 addition & 1 deletion src/core/Geometry.js
Original file line number Diff line number Diff line change
Expand Up @@ -742,7 +742,7 @@ THREE.Geometry.prototype = {

for ( var property in this ) {

delete this[ property ];
this[ property ] = null;

}

Expand Down
2 changes: 1 addition & 1 deletion src/core/Object3D.js
Original file line number Diff line number Diff line change
Expand Up @@ -355,7 +355,7 @@ THREE.Object3D.prototype = {

for ( var property in this ) {

delete this[ property ];
this[ property ] = null;

}

Expand Down
2 changes: 1 addition & 1 deletion src/materials/Material.js
Original file line number Diff line number Diff line change
Expand Up @@ -122,7 +122,7 @@ THREE.Material.prototype.dispose = function () {

for ( var property in this ) {

delete this[ property ];
this[ property ] = null;

}

Expand Down
2 changes: 1 addition & 1 deletion src/textures/Texture.js
Original file line number Diff line number Diff line change
Expand Up @@ -83,7 +83,7 @@ THREE.Texture.prototype = {

for ( var property in this ) {

delete this[ property ];
this[ property ] = null;

}

Expand Down

0 comments on commit 2466896

Please sign in to comment.