Better way to use an object as a dictionary

mbebenita · Jun 26, 2012 · d2488fe · d2488fe
1 parent a3f0dc4
commit d2488fe
Showing 1 changed file with 5 additions and 5 deletions.
diff --git a/src/compiler.js b/src/compiler.js
@@ -273,7 +273,7 @@
     this.name = name;
     this.parent = parent;
     this.root = parent.root;
-    this.variables = {};
+    this.variables = Object.create(null);
     this.frame = parent.frame;
     assert(this.frame instanceof Frame);
   }
@@ -339,7 +339,7 @@
   Scope.prototype.addVariable = function addVariable(variable, external) {
     assert(variable);
     assert(!variable.frame);
-    assert(!this.variables.hasOwnProperty(variable.name));
+    assert(!this.variables[variable.name]);
     variable.frame = this.frame;
 
     var variables = this.variables;
@@ -385,10 +385,10 @@
     this.name = name;
     this.parent = parent;
     this.root = parent ? parent.root : this;
-    this.variables = {};
+    this.variables = Object.create(null);
-    this.cachedLocals = {};
+    this.cachedLocals = Object.create(null);
     this.frame = this;
-    this.mangles = {};
+    this.mangles = Object.create(null);
   }
 
   Frame.prototype = Object.create(Scope.prototype);