Revert "support serialize undefined (yahoo#54)"

This reverts commit 356987f.

Author: okuryu

README.md

@@ -52,7 +52,7 @@ serialize({
 The above will produce the following string output:
 
 ```js
-'{"str":"string","num":0,"obj":{"foo":"foo"},"arr":[1,2,3],"bool":true,"nil":null,"undef":undefined,"date":new Date("2016-04-28T22:02:17.000Z"),"map":new Map([["hello","world"]]),"set":new Set([123,456]),"fn":function echo(arg) { return arg; },"re":/([^\s]+)/g}'
+'{"str":"string","num":0,"obj":{"foo":"foo"},"arr":[1,2,3],"bool":true,"nil":null,date:new Date("2016-04-28T22:02:17.156Z"),new Map([["hello", "world"]]),new Set([123,456]),"fn":function echo(arg) { return arg; },"re":/([^\\s]+)/g}'
 ```
 
 Note: to produced a beautified string, you can pass an optional second argument to `serialize()` to define the number of spaces to be used for the indentation.

index.js

@@ -8,7 +8,7 @@ See the accompanying LICENSE file for terms.
 
 // Generate an internal UID to make the regexp pattern harder to guess.
 var UID                 = Math.floor(Math.random() * 0x10000000000).toString(16);
-var PLACE_HOLDER_REGEXP = new RegExp('"@__(F|R|D|M|S|U)-' + UID + '-(\\d+)__@"', 'g');
+var PLACE_HOLDER_REGEXP = new RegExp('"@__(F|R|D|M|S)-' + UID + '-(\\d+)__@"', 'g');
 
 var IS_NATIVE_CODE_REGEXP = /\{\s*\[native code\]\s*\}/g;
 var IS_PURE_FUNCTION = /function.*?\(/;
@@ -44,13 +44,11 @@ module.exports = function serialize(obj, options) {
     var dates     = [];
     var maps      = [];
     var sets      = [];
-    var undefs    = [];
 
     // Returns placeholders for functions and regexps (identified by index)
     // which are later replaced by their string representation.
     function replacer(key, value) {
-
-        if (!value && value !== undefined) {
+        if (!value) {
             return value;
         }
 
@@ -81,10 +79,6 @@ module.exports = function serialize(obj, options) {
             return '@__F-' + UID + '-' + (functions.push(origValue) - 1) + '__@';
         }
 
-        if (type === 'undefined') {
-            return '@__U-' + UID + '-' + (undefs.push(origValue) - 1) + '__@';
-        }
-
         return value;
     }
 
@@ -125,12 +119,6 @@ module.exports = function serialize(obj, options) {
       return serializedFn;
     }
 
-    // Protects against `JSON.stringify()` returning `undefined`, by serializing
-    // to the literal string: "undefined".
-    if (obj === undefined) {
-        return String(obj);
-    }
-
     var str;
 
     // Creates a JSON string representation of the value.
@@ -154,7 +142,7 @@ module.exports = function serialize(obj, options) {
         str = str.replace(UNSAFE_CHARS_REGEXP, escapeUnsafeChars);
     }
 
-    if (functions.length === 0 && regexps.length === 0 && dates.length === 0 && maps.length === 0 && sets.length === 0 && undefs.length === 0) {
+    if (functions.length === 0 && regexps.length === 0 && dates.length === 0 && maps.length === 0 && sets.length === 0) {
         return str;
     }
 
@@ -178,10 +166,6 @@ module.exports = function serialize(obj, options) {
             return "new Set(" + serialize(Array.from(sets[valueIndex].values()), options) + ")";
         }
 
-        if (type === 'U') {
-            return 'undefined'
-        }
-
         var fn = functions[valueIndex];
 
         return serializeFunc(fn);

test/unit/serialize.js

@@ -57,13 +57,6 @@ describe('serialize( obj )', function () {
             var ws = String.fromCharCode(8232);
             expect(eval(serialize(ws))).to.equal(ws);
         });
-
-        it('should serialize undefined correctly', function () {
-            var obj;
-            var str = '{"undef":undefined,"nest":{"undef":undefined}}';
-            eval('obj = ' + str);
-            expect(serialize(obj)).to.equal(str);
-        });
     });
 
     describe('functions', function () {