Move Object.seal into the Class framework.

creationix · Aug 19, 2010 · e459cb9 · e459cb9
1 parent bcb1fdd
commit e459cb9
Show file tree

Hide file tree

Showing 3 changed files with 2 additions and 5 deletions.
diff --git a/lib/class.js b/lib/class.js
@@ -32,6 +32,8 @@ var Class = module.exports = Object.create(Object.prototype, {
   new: {value: function () {
     var obj = Object.create(this);
     if (obj.initialize) obj.initialize.apply(obj, arguments);
+    // Lock the object down
+    Object.seal(obj);
     return obj;
   }}
 });
diff --git a/lib/nstore.js b/lib/nstore.js
@@ -25,8 +25,6 @@ var nStore = module.exports = Class.extend({
     this.dbLength = 0;
     this.busy = false;
     this.filterFn = null;
-    // We don't want any other properties on this object that aren't initialized here
-    Object.seal(this);
     this.loadDatabase(callback);
   },
 

diff --git a/lib/queue.js b/lib/queue.js
@@ -8,9 +8,6 @@ var Queue = module.exports = Class.extend({
     this.tail = [];
     this.head = Array.prototype.slice.call(arguments);
     this.offset = 0;
-    // Lock the object down
-    Object.seal(this);
-    return this;
   },
   shift: function shift() {
     if (this.offset === this.head.length) {