cleanup

ljharb · Aug 29, 2014 · 83283aa · 83283aa
1 parent 6a97ad1
commit 83283aa
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 22 deletions.
diff --git a/lib/parse.js b/lib/parse.js
@@ -144,7 +144,7 @@ module.exports = function (str, options) {
     // Iterate over the keys and setup the new object
     //
     for (var key in tempObj) {
-        if (Object.prototype.hasOwnProperty.call(tempObj, key)) {
+        if (tempObj.hasOwnProperty(key)) {
             var newObj = internals.parseKeys(key, tempObj[key], options);
             obj = Utils.merge(obj, newObj);
         }

diff --git a/lib/stringify.js b/lib/stringify.js
@@ -32,7 +32,7 @@ internals.stringify = function (obj, prefix) {
     var values = [];
 
     for (var key in obj) {
-        if (Object.prototype.hasOwnProperty.call(obj, key)) {
+        if (obj.hasOwnProperty(key)) {
             values = values.concat(internals.stringify(obj[key], prefix + '[' + key + ']'));
         }
     }
@@ -49,7 +49,7 @@ module.exports = function (obj, options) {
     var keys = [];
 
     for (var key in obj) {
-        if (Object.prototype.hasOwnProperty.call(obj, key)) {
+        if (obj.hasOwnProperty(key)) {
             keys = keys.concat(internals.stringify(obj[key], key));
         }
     }

diff --git a/lib/utils.js b/lib/utils.js
@@ -20,32 +20,17 @@ exports.arrayToObject = function (source) {
 };
 
 
-exports.clone = function (source, refs) {
-
-    if (typeof source !== 'object' ||
-        source === null) {
-
-        return source;
-    }
+exports.clone = function (source) {
 
     if (exports.isBuffer(source)) {
         return source.toString();
     }
 
-    refs = refs || [];
-
-    var lookup = refs.indexOf(source);
-    if (lookup !== -1) {
-        return refs[lookup];
-    }
-
-    var copy = Array.isArray(source) ? [] : source;
-
-    refs.push(source);
+    var copy = Array.isArray(source) ? [] : {};
 
     for (var i in source) {
         if (Object.prototype.hasOwnProperty.call(source, i)) {
-            copy[i] = exports.clone(source[i], refs);
+            copy[i] = source[i];
         }
     }
 
@@ -147,7 +132,7 @@ exports.compact = function (obj, refs) {
     }
 
     for (var key in obj) {
-        if (Object.prototype.hasOwnProperty.call(obj, key)) {
+        if (obj.hasOwnProperty(key)) {
             obj[key] = exports.compact(obj[key], refs);
         }
     }