Use undefined instead of null as the default value (#112)

* Use undefined instead of null as default value

This fixes the type issues in #107. Also it makes more sense. I'm unsure about the `return undefined`.

* Fix tests

Author: Timmmm

compile.js

@@ -101,7 +101,7 @@ function compileDest(ctx) {
     for (var i = 0; i < ctx._proto.fields.length; i++) {
         var field = ctx._proto.fields[i];
         props[field.name + ': ' + JSON.stringify(ctx._defaults[field.name])] = true;
-        if (field.oneof) props[field.oneof + ': null'] = true;
+        if (field.oneof) props[field.oneof + ': undefined'] = true;
     }
     return '{' + Object.keys(props).join(', ') + '}';
 }
@@ -295,7 +295,7 @@ function getDefaultValue(field, value) {
     case 'string':   return value || '';
     case 'bool':     return value === 'true';
     case 'map':      return {};
-    default:         return null;
+    default:         return undefined;
     }
 }
 

test/compile.test.js

@@ -222,8 +222,8 @@ test('handles all implicit default values', function(t) {
     t.equals(data.id, 0);
     t.deepEqual(data.tags, []);
     t.deepEqual(data.numbers, []);
-    t.equals(data.bytes, null);
-    t.equals(data.custom, null);
+    t.equals(data.bytes, undefined);
+    t.equals(data.custom, undefined);
     t.deepEqual(data.types, []);
 
     t.end();
@@ -237,7 +237,7 @@ test('sets oneof field name', function(t) {
     Envelope.write({}, pbf);
     var data = Envelope.read(new Pbf(pbf.finish()));
 
-    t.equals(data.value, null);
+    t.equals(data.value, undefined);
     t.equals(data.id, 0);
 
     pbf = new Pbf();