From 62eef58aa3b002115ebded0fa58acc770cd4e4f4 Mon Sep 17 00:00:00 2001
From: dcodeIO <dcode@dcode.io>
Date: Wed, 4 Jan 2017 12:34:41 +0100
Subject: [PATCH] Fixed: Respect long defaults in converters

---
 src/converter.js                 |   2 +-
 tests/data/ambiguous-names.js    |   6 +-
 tests/data/mapbox/vector_tile.js |  18 +--
 tests/data/package.js            |   6 +-
 tests/data/rpc.js                |   4 +-
 tests/data/test.js               | 186 +++++++++++++++----------------
 6 files changed, 111 insertions(+), 111 deletions(-)

diff --git a/src/converter.js b/src/converter.js
index f58cd1491..abc5ca051 100644
--- a/src/converter.js
+++ b/src/converter.js
@@ -21,7 +21,7 @@ function genConvert(field, fieldIndex, prop) {
         case "fixed64":
         case "sfixed64":
             // longs
-            return sprintf("f.longs(s%s,%d,%d,%j,o)", prop, 0, 0, field.type.charAt(0) === "u");
+            return sprintf("f.longs(s%s,%d,%d,%j,o)", prop, field.typeDefault.low, field.typeDefault.high, field.type.charAt(0) === "u");
         case "bytes":
             // bytes
             return sprintf("f.bytes(s%s,%j,o)", prop, Array.prototype.slice.call(field.typeDefault));
diff --git a/tests/data/ambiguous-names.js b/tests/data/ambiguous-names.js
index fb4ace337..fd97b1612 100644
--- a/tests/data/ambiguous-names.js
+++ b/tests/data/ambiguous-names.js
@@ -134,7 +134,7 @@ $root.A = (function() {
         if (!options) {
             options = {};
         }
-        var dst = impl.create(src, this, options) || null;
+        var dst = impl.create(src, this, options);
         if (dst) {
             if (dst.whatever === undefined && options.defaults) {
                 dst.whatever = "";
@@ -294,9 +294,9 @@ $root.B = (function() {
         if (!options) {
             options = {};
         }
-        var dst = impl.create(src, this, options) || null;
+        var dst = impl.create(src, this, options);
         if (dst) {
-            if (!(src.A === undefined || src.A === null) || options.defaults) {
+            if (src.A !== undefined && src.A !== null || options.defaults) {
                 dst.A = types[0].convert(src.A, impl, options);
             }
         }
diff --git a/tests/data/mapbox/vector_tile.js b/tests/data/mapbox/vector_tile.js
index 38eac17a4..6e8138cb8 100644
--- a/tests/data/mapbox/vector_tile.js
+++ b/tests/data/mapbox/vector_tile.js
@@ -157,7 +157,7 @@ $root.vector_tile = (function() {
             if (!options) {
                 options = {};
             }
-            var dst = impl.create(src, this, options) || null;
+            var dst = impl.create(src, this, options);
             if (dst) {
                 if (src.layers && src.layers.length) {
                     dst.layers = [];
@@ -445,7 +445,7 @@ $root.vector_tile = (function() {
                 if (!options) {
                     options = {};
                 }
-                var dst = impl.create(src, this, options) || null;
+                var dst = impl.create(src, this, options);
                 if (dst) {
                     if (dst.stringValue === undefined && options.defaults) {
                         dst.stringValue = "";
@@ -456,13 +456,13 @@ $root.vector_tile = (function() {
                     if (dst.doubleValue === undefined && options.defaults) {
                         dst.doubleValue = 0;
                     }
-                    if (!(src.intValue === undefined || src.intValue === null) || options.defaults) {
+                    if (src.intValue !== undefined && src.intValue !== null || options.defaults) {
                         dst.intValue = impl.longs(src.intValue, 0, 0, false, options);
                     }
-                    if (!(src.uintValue === undefined || src.uintValue === null) || options.defaults) {
+                    if (src.uintValue !== undefined && src.uintValue !== null || options.defaults) {
                         dst.uintValue = impl.longs(src.uintValue, 0, 0, true, options);
                     }
-                    if (!(src.sintValue === undefined || src.sintValue === null) || options.defaults) {
+                    if (src.sintValue !== undefined && src.sintValue !== null || options.defaults) {
                         dst.sintValue = impl.longs(src.sintValue, 0, 0, false, options);
                     }
                     if (dst.boolValue === undefined && options.defaults) {
@@ -721,9 +721,9 @@ $root.vector_tile = (function() {
                 if (!options) {
                     options = {};
                 }
-                var dst = impl.create(src, this, options) || null;
+                var dst = impl.create(src, this, options);
                 if (dst) {
-                    if (!(src.id === undefined || src.id === null) || options.defaults) {
+                    if (src.id !== undefined && src.id !== null || options.defaults) {
                         dst.id = impl.longs(src.id, 0, 0, true, options);
                     }
                     if (src.tags && src.tags.length) {
@@ -736,7 +736,7 @@ $root.vector_tile = (function() {
                             dst.tags = [];
                         }
                     }
-                    if (!(src.type === undefined || src.type === null) || options.defaults) {
+                    if (src.type !== undefined || options.defaults) {
                         dst.type = impl.enums(src.type, undefined, types[2], options);
                     }
                     if (src.geometry && src.geometry.length) {
@@ -1017,7 +1017,7 @@ $root.vector_tile = (function() {
                 if (!options) {
                     options = {};
                 }
-                var dst = impl.create(src, this, options) || null;
+                var dst = impl.create(src, this, options);
                 if (dst) {
                     if (dst.version === undefined && options.defaults) {
                         dst.version = 1;
diff --git a/tests/data/package.js b/tests/data/package.js
index b740a68f2..5495b7b43 100644
--- a/tests/data/package.js
+++ b/tests/data/package.js
@@ -503,7 +503,7 @@ $root.Package = (function() {
         if (!options) {
             options = {};
         }
-        var dst = impl.create(src, this, options) || null;
+        var dst = impl.create(src, this, options);
         if (dst) {
             if (dst.name === undefined && options.defaults) {
                 dst.name = "";
@@ -520,7 +520,7 @@ $root.Package = (function() {
             if (dst.license === undefined && options.defaults) {
                 dst.license = "";
             }
-            if (!(src.repository === undefined || src.repository === null) || options.defaults) {
+            if (src.repository !== undefined && src.repository !== null || options.defaults) {
                 dst.repository = types[5].convert(src.repository, impl, options);
             }
             if (dst.bugs === undefined && options.defaults) {
@@ -726,7 +726,7 @@ $root.Package = (function() {
             if (!options) {
                 options = {};
             }
-            var dst = impl.create(src, this, options) || null;
+            var dst = impl.create(src, this, options);
             if (dst) {
                 if (dst.type === undefined && options.defaults) {
                     dst.type = "";
diff --git a/tests/data/rpc.js b/tests/data/rpc.js
index 66490b061..46846fffa 100644
--- a/tests/data/rpc.js
+++ b/tests/data/rpc.js
@@ -226,7 +226,7 @@ $root.MyRequest = (function() {
         if (!options) {
             options = {};
         }
-        var dst = impl.create(src, this, options) || null;
+        var dst = impl.create(src, this, options);
         if (dst) {
             if (dst.path === undefined && options.defaults) {
                 dst.path = "";
@@ -382,7 +382,7 @@ $root.MyResponse = (function() {
         if (!options) {
             options = {};
         }
-        var dst = impl.create(src, this, options) || null;
+        var dst = impl.create(src, this, options);
         if (dst) {
             if (dst.status === undefined && options.defaults) {
                 dst.status = 0;
diff --git a/tests/data/test.js b/tests/data/test.js
index 71cdda308..5bd635b9a 100644
--- a/tests/data/test.js
+++ b/tests/data/test.js
@@ -134,7 +134,7 @@ $root.jspb = (function() {
                 if (!options) {
                     options = {};
                 }
-                var dst = impl.create(src, this, options) || null;
+                var dst = impl.create(src, this, options);
                 return dst;
             };})();
 
@@ -309,9 +309,9 @@ $root.jspb = (function() {
                 if (!options) {
                     options = {};
                 }
-                var dst = impl.create(src, this, options) || null;
+                var dst = impl.create(src, this, options);
                 if (dst) {
-                    if (!(src.outerEnum === undefined || src.outerEnum === null) || options.defaults) {
+                    if (src.outerEnum !== undefined || options.defaults) {
                         dst.outerEnum = impl.enums(src.outerEnum, 1, types[0], options);
                     }
                 }
@@ -507,7 +507,7 @@ $root.jspb = (function() {
                 if (!options) {
                     options = {};
                 }
-                var dst = impl.create(src, this, options) || null;
+                var dst = impl.create(src, this, options);
                 if (dst) {
                     if (dst.aString === undefined && options.defaults) {
                         dst.aString = "";
@@ -700,7 +700,7 @@ $root.jspb = (function() {
                 if (!options) {
                     options = {};
                 }
-                var dst = impl.create(src, this, options) || null;
+                var dst = impl.create(src, this, options);
                 if (dst) {
                     if (dst.aString === undefined && options.defaults) {
                         dst.aString = "";
@@ -907,7 +907,7 @@ $root.jspb = (function() {
                 if (!options) {
                     options = {};
                 }
-                var dst = impl.create(src, this, options) || null;
+                var dst = impl.create(src, this, options);
                 if (dst) {
                     if (dst.normal === undefined && options.defaults) {
                         dst.normal = "";
@@ -1165,7 +1165,7 @@ $root.jspb = (function() {
                 if (!options) {
                     options = {};
                 }
-                var dst = impl.create(src, this, options) || null;
+                var dst = impl.create(src, this, options);
                 if (dst) {
                     if (dst.aString === undefined && options.defaults) {
                         dst.aString = "";
@@ -1173,7 +1173,7 @@ $root.jspb = (function() {
                     if (dst.aBool === undefined && options.defaults) {
                         dst.aBool = false;
                     }
-                    if (!(src.aNestedMessage === undefined || src.aNestedMessage === null) || options.defaults) {
+                    if (src.aNestedMessage !== undefined && src.aNestedMessage !== null || options.defaults) {
                         dst.aNestedMessage = types[2].convert(src.aNestedMessage, impl, options);
                     }
                     if (src.aRepeatedMessage && src.aRepeatedMessage.length) {
@@ -1344,7 +1344,7 @@ $root.jspb = (function() {
                     if (!options) {
                         options = {};
                     }
-                    var dst = impl.create(src, this, options) || null;
+                    var dst = impl.create(src, this, options);
                     if (dst) {
                         if (dst.anInt === undefined && options.defaults) {
                             dst.anInt = 0;
@@ -1680,7 +1680,7 @@ $root.jspb = (function() {
                 if (!options) {
                     options = {};
                 }
-                var dst = impl.create(src, this, options) || null;
+                var dst = impl.create(src, this, options);
                 if (dst) {
                     if (dst.str1 === undefined && options.defaults) {
                         dst.str1 = "";
@@ -1691,10 +1691,10 @@ $root.jspb = (function() {
                     if (dst.str3 === undefined && options.defaults) {
                         dst.str3 = "";
                     }
-                    if (!(src[".jspb.test.IsExtension.extField"] === undefined || src[".jspb.test.IsExtension.extField"] === null) || options.defaults) {
+                    if (src[".jspb.test.IsExtension.extField"] !== undefined && src[".jspb.test.IsExtension.extField"] !== null || options.defaults) {
                         dst[".jspb.test.IsExtension.extField"] = types[3].convert(src[".jspb.test.IsExtension.extField"], impl, options);
                     }
-                    if (!(src[".jspb.test.IndirectExtension.simple"] === undefined || src[".jspb.test.IndirectExtension.simple"] === null) || options.defaults) {
+                    if (src[".jspb.test.IndirectExtension.simple"] !== undefined && src[".jspb.test.IndirectExtension.simple"] !== null || options.defaults) {
                         dst[".jspb.test.IndirectExtension.simple"] = types[4].convert(src[".jspb.test.IndirectExtension.simple"], impl, options);
                     }
                     if (dst[".jspb.test.IndirectExtension.str"] === undefined && options.defaults) {
@@ -1720,7 +1720,7 @@ $root.jspb = (function() {
                             dst[".jspb.test.IndirectExtension.repeatedSimple"] = [];
                         }
                     }
-                    if (!(src[".jspb.test.simple1"] === undefined || src[".jspb.test.simple1"] === null) || options.defaults) {
+                    if (src[".jspb.test.simple1"] !== undefined && src[".jspb.test.simple1"] !== null || options.defaults) {
                         dst[".jspb.test.simple1"] = types[8].convert(src[".jspb.test.simple1"], impl, options);
                     }
                 }
@@ -1963,7 +1963,7 @@ $root.jspb = (function() {
                 if (!options) {
                     options = {};
                 }
-                var dst = impl.create(src, this, options) || null;
+                var dst = impl.create(src, this, options);
                 if (dst) {
                     if (dst.aString === undefined && options.defaults) {
                         dst.aString = "";
@@ -1971,7 +1971,7 @@ $root.jspb = (function() {
                     if (dst.anOutOfOrderBool === undefined && options.defaults) {
                         dst.anOutOfOrderBool = false;
                     }
-                    if (!(src.aNestedMessage === undefined || src.aNestedMessage === null) || options.defaults) {
+                    if (src.aNestedMessage !== undefined && src.aNestedMessage !== null || options.defaults) {
                         dst.aNestedMessage = types[2].convert(src.aNestedMessage, impl, options);
                     }
                     if (src.aRepeatedMessage && src.aRepeatedMessage.length) {
@@ -2138,7 +2138,7 @@ $root.jspb = (function() {
                     if (!options) {
                         options = {};
                     }
-                    var dst = impl.create(src, this, options) || null;
+                    var dst = impl.create(src, this, options);
                     if (dst) {
                         if (dst.anInt === undefined && options.defaults) {
                             dst.anInt = 0;
@@ -2279,7 +2279,7 @@ $root.jspb = (function() {
                 if (!options) {
                     options = {};
                 }
-                var dst = impl.create(src, this, options) || null;
+                var dst = impl.create(src, this, options);
                 return dst;
             };})();
 
@@ -2427,7 +2427,7 @@ $root.jspb = (function() {
                     if (!options) {
                         options = {};
                     }
-                    var dst = impl.create(src, this, options) || null;
+                    var dst = impl.create(src, this, options);
                     if (dst) {
                         if (dst.innerComplexField === undefined && options.defaults) {
                             dst.innerComplexField = 0;
@@ -2586,7 +2586,7 @@ $root.jspb = (function() {
                 if (!options) {
                     options = {};
                 }
-                var dst = impl.create(src, this, options) || null;
+                var dst = impl.create(src, this, options);
                 if (dst) {
                     if (dst.ext1 === undefined && options.defaults) {
                         dst.ext1 = "";
@@ -2724,7 +2724,7 @@ $root.jspb = (function() {
                 if (!options) {
                     options = {};
                 }
-                var dst = impl.create(src, this, options) || null;
+                var dst = impl.create(src, this, options);
                 return dst;
             };})();
 
@@ -2973,7 +2973,7 @@ $root.jspb = (function() {
                 if (!options) {
                     options = {};
                 }
-                var dst = impl.create(src, this, options) || null;
+                var dst = impl.create(src, this, options);
                 if (dst) {
                     if (dst.stringField === undefined && options.defaults) {
                         dst.stringField = "default<>'\"abc";
@@ -2981,16 +2981,16 @@ $root.jspb = (function() {
                     if (dst.boolField === undefined && options.defaults) {
                         dst.boolField = true;
                     }
-                    if (!(src.intField === undefined || src.intField === null) || options.defaults) {
-                        dst.intField = impl.longs(src.intField, 0, 0, false, options);
+                    if (src.intField !== undefined && src.intField !== null || options.defaults) {
+                        dst.intField = impl.longs(src.intField, 11, 0, false, options);
                     }
-                    if (!(src.enumField === undefined || src.enumField === null) || options.defaults) {
+                    if (src.enumField !== undefined || options.defaults) {
                         dst.enumField = impl.enums(src.enumField, undefined, types[3], options);
                     }
                     if (dst.emptyField === undefined && options.defaults) {
                         dst.emptyField = "";
                     }
-                    if (!(src.bytesField === undefined || src.bytesField === null) || options.defaults) {
+                    if (src.bytesField !== undefined || options.defaults) {
                         dst.bytesField = impl.bytes(src.bytesField, [ 109, 111, 111 ], options);
                     }
                 }
@@ -3298,7 +3298,7 @@ $root.jspb = (function() {
                 if (!options) {
                     options = {};
                 }
-                var dst = impl.create(src, this, options) || null;
+                var dst = impl.create(src, this, options);
                 if (dst) {
                     if (dst.optionalFloatField === undefined && options.defaults) {
                         dst.optionalFloatField = 0;
@@ -3596,12 +3596,12 @@ $root.jspb = (function() {
                 if (!options) {
                     options = {};
                 }
-                var dst = impl.create(src, this, options) || null;
+                var dst = impl.create(src, this, options);
                 if (dst) {
                     if (dst.str === undefined && options.defaults) {
                         dst.str = "";
                     }
-                    if (!(src.simple1 === undefined || src.simple1 === null) || options.defaults) {
+                    if (src.simple1 !== undefined && src.simple1 !== null || options.defaults) {
                         dst.simple1 = types[1].convert(src.simple1, impl, options);
                     }
                     if (src.simple2 && src.simple2.length) {
@@ -3614,13 +3614,13 @@ $root.jspb = (function() {
                             dst.simple2 = [];
                         }
                     }
-                    if (!(src.bytesField === undefined || src.bytesField === null) || options.defaults) {
+                    if (src.bytesField !== undefined || options.defaults) {
                         dst.bytesField = impl.bytes(src.bytesField, [], options);
                     }
                     if (dst.unused === undefined && options.defaults) {
                         dst.unused = "";
                     }
-                    if (!(src[".jspb.test.CloneExtension.extField"] === undefined || src[".jspb.test.CloneExtension.extField"] === null) || options.defaults) {
+                    if (src[".jspb.test.CloneExtension.extField"] !== undefined && src[".jspb.test.CloneExtension.extField"] !== null || options.defaults) {
                         dst[".jspb.test.CloneExtension.extField"] = types[5].convert(src[".jspb.test.CloneExtension.extField"], impl, options);
                     }
                 }
@@ -3774,7 +3774,7 @@ $root.jspb = (function() {
                 if (!options) {
                     options = {};
                 }
-                var dst = impl.create(src, this, options) || null;
+                var dst = impl.create(src, this, options);
                 if (dst) {
                     if (dst.ext === undefined && options.defaults) {
                         dst.ext = "";
@@ -3967,15 +3967,15 @@ $root.jspb = (function() {
                 if (!options) {
                     options = {};
                 }
-                var dst = impl.create(src, this, options) || null;
+                var dst = impl.create(src, this, options);
                 if (dst) {
                     if (dst.id === undefined && options.defaults) {
                         dst.id = "";
                     }
-                    if (!(src.requiredSimple === undefined || src.requiredSimple === null) || options.defaults) {
+                    if (src.requiredSimple !== undefined && src.requiredSimple !== null || options.defaults) {
                         dst.requiredSimple = types[1].convert(src.requiredSimple, impl, options);
                     }
-                    if (!(src.optionalSimple === undefined || src.optionalSimple === null) || options.defaults) {
+                    if (src.optionalSimple !== undefined && src.optionalSimple !== null || options.defaults) {
                         dst.optionalSimple = types[2].convert(src.optionalSimple, impl, options);
                     }
                 }
@@ -4148,7 +4148,7 @@ $root.jspb = (function() {
                 if (!options) {
                     options = {};
                 }
-                var dst = impl.create(src, this, options) || null;
+                var dst = impl.create(src, this, options);
                 if (dst) {
                     if (dst.extension === undefined && options.defaults) {
                         dst.extension = 0;
@@ -4289,7 +4289,7 @@ $root.jspb = (function() {
                 if (!options) {
                     options = {};
                 }
-                var dst = impl.create(src, this, options) || null;
+                var dst = impl.create(src, this, options);
                 return dst;
             };})();
 
@@ -4712,7 +4712,7 @@ $root.jspb = (function() {
                 if (!options) {
                     options = {};
                 }
-                var dst = impl.create(src, this, options) || null;
+                var dst = impl.create(src, this, options);
                 if (dst) {
                     if (dst.pone === undefined && options.defaults) {
                         dst.pone = "";
@@ -4720,7 +4720,7 @@ $root.jspb = (function() {
                     if (dst.pthree === undefined && options.defaults) {
                         dst.pthree = "";
                     }
-                    if (!(src.rone === undefined || src.rone === null) || options.defaults) {
+                    if (src.rone !== undefined && src.rone !== null || options.defaults) {
                         dst.rone = types[2].convert(src.rone, impl, options);
                     }
                     if (dst.rtwo === undefined && options.defaults) {
@@ -4920,12 +4920,12 @@ $root.jspb = (function() {
                 if (!options) {
                     options = {};
                 }
-                var dst = impl.create(src, this, options) || null;
+                var dst = impl.create(src, this, options);
                 if (dst) {
                     if (dst.value === undefined && options.defaults) {
                         dst.value = 0;
                     }
-                    if (!(src.data === undefined || src.data === null) || options.defaults) {
+                    if (src.data !== undefined || options.defaults) {
                         dst.data = impl.bytes(src.data, [], options);
                     }
                 }
@@ -5487,7 +5487,7 @@ $root.jspb = (function() {
                 if (!options) {
                     options = {};
                 }
-                var dst = impl.create(src, this, options) || null;
+                var dst = impl.create(src, this, options);
                 if (dst) {
                     if (dst.mapStringString === undefined && options.defaults) {
                         dst.mapStringString = "";
@@ -5495,7 +5495,7 @@ $root.jspb = (function() {
                     if (dst.mapStringInt32 === undefined && options.defaults) {
                         dst.mapStringInt32 = 0;
                     }
-                    if (!(src.mapStringInt64 === undefined || src.mapStringInt64 === null) || options.defaults) {
+                    if (src.mapStringInt64 !== undefined && src.mapStringInt64 !== null || options.defaults) {
                         dst.mapStringInt64 = impl.longs(src.mapStringInt64, 0, 0, false, options);
                     }
                     if (dst.mapStringBool === undefined && options.defaults) {
@@ -5504,10 +5504,10 @@ $root.jspb = (function() {
                     if (dst.mapStringDouble === undefined && options.defaults) {
                         dst.mapStringDouble = 0;
                     }
-                    if (!(src.mapStringEnum === undefined || src.mapStringEnum === null) || options.defaults) {
+                    if (src.mapStringEnum !== undefined || options.defaults) {
                         dst.mapStringEnum = impl.enums(src.mapStringEnum, 0, types[5], options);
                     }
-                    if (!(src.mapStringMsg === undefined || src.mapStringMsg === null) || options.defaults) {
+                    if (src.mapStringMsg !== undefined && src.mapStringMsg !== null || options.defaults) {
                         dst.mapStringMsg = types[6].convert(src.mapStringMsg, impl, options);
                     }
                     if (dst.mapInt32String === undefined && options.defaults) {
@@ -5519,10 +5519,10 @@ $root.jspb = (function() {
                     if (dst.mapBoolString === undefined && options.defaults) {
                         dst.mapBoolString = "";
                     }
-                    if (!(src.testMapFields === undefined || src.testMapFields === null) || options.defaults) {
+                    if (src.testMapFields !== undefined && src.testMapFields !== null || options.defaults) {
                         dst.testMapFields = types[10].convert(src.testMapFields, impl, options);
                     }
-                    if (!(src.mapStringTestmapfields === undefined || src.mapStringTestmapfields === null) || options.defaults) {
+                    if (src.mapStringTestmapfields !== undefined && src.mapStringTestmapfields !== null || options.defaults) {
                         dst.mapStringTestmapfields = types[11].convert(src.mapStringTestmapfields, impl, options);
                     }
                 }
@@ -5694,7 +5694,7 @@ $root.jspb = (function() {
                 if (!options) {
                     options = {};
                 }
-                var dst = impl.create(src, this, options) || null;
+                var dst = impl.create(src, this, options);
                 if (dst) {
                     if (dst.foo === undefined && options.defaults) {
                         dst.foo = 0;
@@ -5832,7 +5832,7 @@ $root.jspb = (function() {
                 if (!options) {
                     options = {};
                 }
-                var dst = impl.create(src, this, options) || null;
+                var dst = impl.create(src, this, options);
                 return dst;
             };})();
 
@@ -5962,7 +5962,7 @@ $root.jspb = (function() {
                     if (!options) {
                         options = {};
                     }
-                    var dst = impl.create(src, this, options) || null;
+                    var dst = impl.create(src, this, options);
                     return dst;
                 };})();
 
@@ -6110,7 +6110,7 @@ $root.jspb = (function() {
                         if (!options) {
                             options = {};
                         }
-                        var dst = impl.create(src, this, options) || null;
+                        var dst = impl.create(src, this, options);
                         if (dst) {
                             if (dst.count === undefined && options.defaults) {
                                 dst.count = 0;
@@ -6310,7 +6310,7 @@ $root.google = (function() {
                 if (!options) {
                     options = {};
                 }
-                var dst = impl.create(src, this, options) || null;
+                var dst = impl.create(src, this, options);
                 if (dst) {
                     if (src.file && src.file.length) {
                         dst.file = [];
@@ -6751,7 +6751,7 @@ $root.google = (function() {
                 if (!options) {
                     options = {};
                 }
-                var dst = impl.create(src, this, options) || null;
+                var dst = impl.create(src, this, options);
                 if (dst) {
                     if (dst.name === undefined && options.defaults) {
                         dst.name = "";
@@ -6829,10 +6829,10 @@ $root.google = (function() {
                             dst.extension = [];
                         }
                     }
-                    if (!(src.options === undefined || src.options === null) || options.defaults) {
+                    if (src.options !== undefined && src.options !== null || options.defaults) {
                         dst.options = types[9].convert(src.options, impl, options);
                     }
-                    if (!(src.sourceCodeInfo === undefined || src.sourceCodeInfo === null) || options.defaults) {
+                    if (src.sourceCodeInfo !== undefined && src.sourceCodeInfo !== null || options.defaults) {
                         dst.sourceCodeInfo = types[10].convert(src.sourceCodeInfo, impl, options);
                     }
                     if (dst.syntax === undefined && options.defaults) {
@@ -7242,7 +7242,7 @@ $root.google = (function() {
                 if (!options) {
                     options = {};
                 }
-                var dst = impl.create(src, this, options) || null;
+                var dst = impl.create(src, this, options);
                 if (dst) {
                     if (dst.name === undefined && options.defaults) {
                         dst.name = "";
@@ -7307,7 +7307,7 @@ $root.google = (function() {
                             dst.oneofDecl = [];
                         }
                     }
-                    if (!(src.options === undefined || src.options === null) || options.defaults) {
+                    if (src.options !== undefined && src.options !== null || options.defaults) {
                         dst.options = types[7].convert(src.options, impl, options);
                     }
                     if (src.reservedRange && src.reservedRange.length) {
@@ -7496,7 +7496,7 @@ $root.google = (function() {
                     if (!options) {
                         options = {};
                     }
-                    var dst = impl.create(src, this, options) || null;
+                    var dst = impl.create(src, this, options);
                     if (dst) {
                         if (dst.start === undefined && options.defaults) {
                             dst.start = 0;
@@ -7673,7 +7673,7 @@ $root.google = (function() {
                     if (!options) {
                         options = {};
                     }
-                    var dst = impl.create(src, this, options) || null;
+                    var dst = impl.create(src, this, options);
                     if (dst) {
                         if (dst.start === undefined && options.defaults) {
                             dst.start = 0;
@@ -8028,7 +8028,7 @@ $root.google = (function() {
                 if (!options) {
                     options = {};
                 }
-                var dst = impl.create(src, this, options) || null;
+                var dst = impl.create(src, this, options);
                 if (dst) {
                     if (dst.name === undefined && options.defaults) {
                         dst.name = "";
@@ -8036,10 +8036,10 @@ $root.google = (function() {
                     if (dst.number === undefined && options.defaults) {
                         dst.number = 0;
                     }
-                    if (!(src.label === undefined || src.label === null) || options.defaults) {
+                    if (src.label !== undefined || options.defaults) {
                         dst.label = impl.enums(src.label, 1, types[2], options);
                     }
-                    if (!(src.type === undefined || src.type === null) || options.defaults) {
+                    if (src.type !== undefined || options.defaults) {
                         dst.type = impl.enums(src.type, 1, types[3].values, options);
                     }
                     if (dst.typeName === undefined && options.defaults) {
@@ -8057,7 +8057,7 @@ $root.google = (function() {
                     if (dst.jsonName === undefined && options.defaults) {
                         dst.jsonName = "";
                     }
-                    if (!(src.options === undefined || src.options === null) || options.defaults) {
+                    if (src.options !== undefined && src.options !== null || options.defaults) {
                         dst.options = types[9].convert(src.options, impl, options);
                     }
                 }
@@ -8299,12 +8299,12 @@ $root.google = (function() {
                 if (!options) {
                     options = {};
                 }
-                var dst = impl.create(src, this, options) || null;
+                var dst = impl.create(src, this, options);
                 if (dst) {
                     if (dst.name === undefined && options.defaults) {
                         dst.name = "";
                     }
-                    if (!(src.options === undefined || src.options === null) || options.defaults) {
+                    if (src.options !== undefined && src.options !== null || options.defaults) {
                         dst.options = types[1].convert(src.options, impl, options);
                     }
                 }
@@ -8509,7 +8509,7 @@ $root.google = (function() {
                 if (!options) {
                     options = {};
                 }
-                var dst = impl.create(src, this, options) || null;
+                var dst = impl.create(src, this, options);
                 if (dst) {
                     if (dst.name === undefined && options.defaults) {
                         dst.name = "";
@@ -8524,7 +8524,7 @@ $root.google = (function() {
                             dst.value = [];
                         }
                     }
-                    if (!(src.options === undefined || src.options === null) || options.defaults) {
+                    if (src.options !== undefined && src.options !== null || options.defaults) {
                         dst.options = types[2].convert(src.options, impl, options);
                     }
                 }
@@ -8718,7 +8718,7 @@ $root.google = (function() {
                 if (!options) {
                     options = {};
                 }
-                var dst = impl.create(src, this, options) || null;
+                var dst = impl.create(src, this, options);
                 if (dst) {
                     if (dst.name === undefined && options.defaults) {
                         dst.name = "";
@@ -8726,7 +8726,7 @@ $root.google = (function() {
                     if (dst.number === undefined && options.defaults) {
                         dst.number = 0;
                     }
-                    if (!(src.options === undefined || src.options === null) || options.defaults) {
+                    if (src.options !== undefined && src.options !== null || options.defaults) {
                         dst.options = types[2].convert(src.options, impl, options);
                     }
                 }
@@ -8931,7 +8931,7 @@ $root.google = (function() {
                 if (!options) {
                     options = {};
                 }
-                var dst = impl.create(src, this, options) || null;
+                var dst = impl.create(src, this, options);
                 if (dst) {
                     if (dst.name === undefined && options.defaults) {
                         dst.name = "";
@@ -8946,7 +8946,7 @@ $root.google = (function() {
                             dst.method = [];
                         }
                     }
-                    if (!(src.options === undefined || src.options === null) || options.defaults) {
+                    if (src.options !== undefined && src.options !== null || options.defaults) {
                         dst.options = types[2].convert(src.options, impl, options);
                     }
                 }
@@ -9194,7 +9194,7 @@ $root.google = (function() {
                 if (!options) {
                     options = {};
                 }
-                var dst = impl.create(src, this, options) || null;
+                var dst = impl.create(src, this, options);
                 if (dst) {
                     if (dst.name === undefined && options.defaults) {
                         dst.name = "";
@@ -9205,7 +9205,7 @@ $root.google = (function() {
                     if (dst.outputType === undefined && options.defaults) {
                         dst.outputType = "";
                     }
-                    if (!(src.options === undefined || src.options === null) || options.defaults) {
+                    if (src.options !== undefined && src.options !== null || options.defaults) {
                         dst.options = types[3].convert(src.options, impl, options);
                     }
                     if (dst.clientStreaming === undefined && options.defaults) {
@@ -9637,7 +9637,7 @@ $root.google = (function() {
                 if (!options) {
                     options = {};
                 }
-                var dst = impl.create(src, this, options) || null;
+                var dst = impl.create(src, this, options);
                 if (dst) {
                     if (dst.javaPackage === undefined && options.defaults) {
                         dst.javaPackage = "";
@@ -9654,7 +9654,7 @@ $root.google = (function() {
                     if (dst.javaStringCheckUtf8 === undefined && options.defaults) {
                         dst.javaStringCheckUtf8 = false;
                     }
-                    if (!(src.optimizeFor === undefined || src.optimizeFor === null) || options.defaults) {
+                    if (src.optimizeFor !== undefined || options.defaults) {
                         dst.optimizeFor = impl.enums(src.optimizeFor, undefined, types[5], options);
                     }
                     if (dst.goPackage === undefined && options.defaults) {
@@ -9946,7 +9946,7 @@ $root.google = (function() {
                 if (!options) {
                     options = {};
                 }
-                var dst = impl.create(src, this, options) || null;
+                var dst = impl.create(src, this, options);
                 if (dst) {
                     if (dst.messageSetWireFormat === undefined && options.defaults) {
                         dst.messageSetWireFormat = false;
@@ -10255,15 +10255,15 @@ $root.google = (function() {
                 if (!options) {
                     options = {};
                 }
-                var dst = impl.create(src, this, options) || null;
+                var dst = impl.create(src, this, options);
                 if (dst) {
-                    if (!(src.ctype === undefined || src.ctype === null) || options.defaults) {
+                    if (src.ctype !== undefined || options.defaults) {
                         dst.ctype = impl.enums(src.ctype, undefined, types[0], options);
                     }
                     if (dst.packed === undefined && options.defaults) {
                         dst.packed = false;
                     }
-                    if (!(src.jstype === undefined || src.jstype === null) || options.defaults) {
+                    if (src.jstype !== undefined || options.defaults) {
                         dst.jstype = impl.enums(src.jstype, undefined, types[2].values, options);
                     }
                     if (dst.lazy === undefined && options.defaults) {
@@ -10486,7 +10486,7 @@ $root.google = (function() {
                 if (!options) {
                     options = {};
                 }
-                var dst = impl.create(src, this, options) || null;
+                var dst = impl.create(src, this, options);
                 if (dst) {
                     if (src.uninterpretedOption && src.uninterpretedOption.length) {
                         dst.uninterpretedOption = [];
@@ -10718,7 +10718,7 @@ $root.google = (function() {
                 if (!options) {
                     options = {};
                 }
-                var dst = impl.create(src, this, options) || null;
+                var dst = impl.create(src, this, options);
                 if (dst) {
                     if (dst.allowAlias === undefined && options.defaults) {
                         dst.allowAlias = false;
@@ -10922,7 +10922,7 @@ $root.google = (function() {
                 if (!options) {
                     options = {};
                 }
-                var dst = impl.create(src, this, options) || null;
+                var dst = impl.create(src, this, options);
                 if (dst) {
                     if (dst.deprecated === undefined && options.defaults) {
                         dst.deprecated = false;
@@ -11120,7 +11120,7 @@ $root.google = (function() {
                 if (!options) {
                     options = {};
                 }
-                var dst = impl.create(src, this, options) || null;
+                var dst = impl.create(src, this, options);
                 if (dst) {
                     if (dst.deprecated === undefined && options.defaults) {
                         dst.deprecated = false;
@@ -11342,12 +11342,12 @@ $root.google = (function() {
                 if (!options) {
                     options = {};
                 }
-                var dst = impl.create(src, this, options) || null;
+                var dst = impl.create(src, this, options);
                 if (dst) {
                     if (dst.deprecated === undefined && options.defaults) {
                         dst.deprecated = false;
                     }
-                    if (!(src.idempotencyLevel === undefined || src.idempotencyLevel === null) || options.defaults) {
+                    if (src.idempotencyLevel !== undefined || options.defaults) {
                         dst.idempotencyLevel = impl.enums(src.idempotencyLevel, undefined, types[1], options);
                     }
                     if (src.uninterpretedOption && src.uninterpretedOption.length) {
@@ -11651,7 +11651,7 @@ $root.google = (function() {
                 if (!options) {
                     options = {};
                 }
-                var dst = impl.create(src, this, options) || null;
+                var dst = impl.create(src, this, options);
                 if (dst) {
                     if (src.name && src.name.length) {
                         dst.name = [];
@@ -11666,16 +11666,16 @@ $root.google = (function() {
                     if (dst.identifierValue === undefined && options.defaults) {
                         dst.identifierValue = "";
                     }
-                    if (!(src.positiveIntValue === undefined || src.positiveIntValue === null) || options.defaults) {
+                    if (src.positiveIntValue !== undefined && src.positiveIntValue !== null || options.defaults) {
                         dst.positiveIntValue = impl.longs(src.positiveIntValue, 0, 0, true, options);
                     }
-                    if (!(src.negativeIntValue === undefined || src.negativeIntValue === null) || options.defaults) {
+                    if (src.negativeIntValue !== undefined && src.negativeIntValue !== null || options.defaults) {
                         dst.negativeIntValue = impl.longs(src.negativeIntValue, 0, 0, false, options);
                     }
                     if (dst.doubleValue === undefined && options.defaults) {
                         dst.doubleValue = 0;
                     }
-                    if (!(src.stringValue === undefined || src.stringValue === null) || options.defaults) {
+                    if (src.stringValue !== undefined || options.defaults) {
                         dst.stringValue = impl.bytes(src.stringValue, [], options);
                     }
                     if (dst.aggregateValue === undefined && options.defaults) {
@@ -11839,7 +11839,7 @@ $root.google = (function() {
                     if (!options) {
                         options = {};
                     }
-                    var dst = impl.create(src, this, options) || null;
+                    var dst = impl.create(src, this, options);
                     if (dst) {
                         if (dst.namePart === undefined && options.defaults) {
                             dst.namePart = "";
@@ -12015,7 +12015,7 @@ $root.google = (function() {
                 if (!options) {
                     options = {};
                 }
-                var dst = impl.create(src, this, options) || null;
+                var dst = impl.create(src, this, options);
                 if (dst) {
                     if (src.location && src.location.length) {
                         dst.location = [];
@@ -12295,7 +12295,7 @@ $root.google = (function() {
                     if (!options) {
                         options = {};
                     }
-                    var dst = impl.create(src, this, options) || null;
+                    var dst = impl.create(src, this, options);
                     if (dst) {
                         if (src.path && src.path.length) {
                             dst.path = [];
@@ -12501,7 +12501,7 @@ $root.google = (function() {
                 if (!options) {
                     options = {};
                 }
-                var dst = impl.create(src, this, options) || null;
+                var dst = impl.create(src, this, options);
                 if (dst) {
                     if (src.annotation && src.annotation.length) {
                         dst.annotation = [];
@@ -12734,7 +12734,7 @@ $root.google = (function() {
                     if (!options) {
                         options = {};
                     }
-                    var dst = impl.create(src, this, options) || null;
+                    var dst = impl.create(src, this, options);
                     if (dst) {
                         if (src.path && src.path.length) {
                             dst.path = [];