Skip defining getters and setters on IE8 entirely, automate defining fallbacks

Author: dcodeIO

dist/protobuf.js

@@ -35,7 +35,7 @@ function Enum(name, values, options) {
     this._valuesById = null;
 }
 
-Object.defineProperties(EnumPrototype, {
+util.props(EnumPrototype, {
 
     /**
      * Enum values by id.
@@ -44,7 +44,7 @@ Object.defineProperties(EnumPrototype, {
      * @readonly
      */
     valuesById: {
-        get: EnumPrototype.getValuesById = function getValuesById() {
+        get: function getValuesById() {
             if (!this._valuesById) {
                 this._valuesById = {};
                 Object.keys(this.values).forEach(function(name) {

dist/protobuf.js.map

@@ -141,7 +141,7 @@ function Field(name, id, type, rule, extend, options) {
     this._packed = null;
 }
 
-Object.defineProperties(FieldPrototype, {
+util.props(FieldPrototype, {
 
     /**
      * Determines whether this field is packed. Only relevant when repeated and working with proto2.

dist/protobuf.min.js

@@ -124,7 +124,7 @@ function inherits(clazz, type, options) {
 
         }, true);
 
-    Object.defineProperties(clazz, classProperties);
+    util.props(clazz, classProperties);
     var prototype = inherits.defineProperties(new Prototype(), type);
     clazz.prototype = prototype;
     prototype.constructor = clazz;
@@ -167,9 +167,8 @@ inherits.defineProperties = function defineProperties(prototype, type) {
 
     // Define each oneof with a non-enumerable getter and setter for the present field
     type.getOneofsArray().forEach(function(oneof) {
-        oneof.resolve();
-        prototypeProperties[oneof.name] = {
-            get: prototype['get' + oneof.ucName] = function() {
+        util.prop(prototype, oneof.resolve().name, {
+            get: function getVirtual() {
                 var keys = oneof.oneof;
                 for (var i = 0; i < keys.length; ++i) {
                     var field = oneof.parent.fields[keys[i]];
@@ -178,16 +177,16 @@ inherits.defineProperties = function defineProperties(prototype, type) {
                 }
                 return undefined;
             },
-            set: prototype['set' + oneof.ucName] = function(value) {
+            set: function setVirtual(value) {
                 var keys = oneof.oneof;
                 for (var i = 0; i < keys.length; ++i) {
                     if (keys[i] !== value)
                         delete this[keys[i]];
                 }
             }
-        };
+        });
     });
 
-    Object.defineProperties(prototype, prototypeProperties);
+    util.props(prototype, prototypeProperties);
     return prototype;
 };

dist/protobuf.min.js.gz

@@ -46,7 +46,7 @@ function clearCache(namespace) {
     return namespace;
 }
 
-Object.defineProperties(NamespacePrototype, {
+util.props(NamespacePrototype, {
 
     /**
      * Nested objects of this namespace as an array for iteration.
@@ -55,7 +55,7 @@ Object.defineProperties(NamespacePrototype, {
      * @readonly
      */
     nestedArray: {
-        get: NamespacePrototype.getNestedArray = function getNestedArray() {
+        get: function getNestedArray() {
             return this._nestedArray || (this._nestedArray = util.toArray(this.nested));
         }
     }

dist/protobuf.min.js.map

@@ -50,7 +50,7 @@ function ReflectionObject(name, options) {
 /** @alias ReflectionObject.prototype */
 var ReflectionObjectPrototype = ReflectionObject.prototype;
 
-Object.defineProperties(ReflectionObjectPrototype, {
+util.props(ReflectionObjectPrototype, {
 
     /**
      * Reference to the root namespace.
@@ -59,7 +59,7 @@ Object.defineProperties(ReflectionObjectPrototype, {
      * @readonly
      */
     root: {
-        get: ReflectionObjectPrototype.getRoot = function getRoot() {
+        get: function getRoot() {
             var ptr = this;
             while (ptr.parent !== null)
                 ptr = ptr.parent;