protobufjs 4.1.2 → 5.0.2

package/index.js

@@ -1,18 +1 @@
-/*
- Copyright 2013 Daniel Wirtz <dcode@dcode.io>
-
- Licensed under the Apache License, Version 2.0 (the "License");
- you may not use this file except in compliance with the License.
- You may obtain a copy of the License at
-
- http://www.apache.org/licenses/LICENSE-2.0
-
- Unless required by applicable law or agreed to in writing, software
- distributed under the License is distributed on an "AS IS" BASIS,
- WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
- See the License for the specific language governing permissions and
- limitations under the License.
- */
-var ProtoBuf = require("./dist/ProtoBuf.js");
-
-module.exports = ProtoBuf;
+module.exports = require("./dist/protobuf.js");

package/jsdoc.json

@@ -3,7 +3,7 @@
         "allowUnknownTags": true
     },
     "source": {
-        "include": ["dist/ProtoBuf.js"]
+        "include": ["dist/protobuf.js"]
     },
     "opts": {
         "destination": "docs"

package/package.json

@@ -1,23 +1,23 @@
 {
   "name": "protobufjs",
-  "version": "4.1.2",
+  "version": "5.0.2",
   "description": "Protocol Buffers for JavaScript. Finally.",
   "author": "Daniel Wirtz <dcode@dcode.io>",
   "contributors": [
     "Frank Xu <yyfrankyy@gmail.com>"
   ],
-  "main": "dist/ProtoBuf.js",
+  "main": "./dist/protobuf.js",
   "bin": {
     "pbjs": "./bin/pbjs"
   },
   "repository": {
     "type": "git",
-    "url": "https://github.com/dcodeIO/ProtoBuf.js.git"
+    "url": "https://github.com/dcodeIO/protobuf.js.git"
   },
   "bugs": {
-    "url": "https://github.com/dcodeIO/ProtoBuf.js/issues"
+    "url": "https://github.com/dcodeIO/protobuf.js/issues"
   },
-  "homepage": "https://github.com/dcodeIO/ProtoBuf.js",
+  "homepage": "https://github.com/dcodeIO/protobuf.js",
   "keywords": [
     "net",
     "buffer",
@@ -29,8 +29,8 @@
   ],
   "dependencies": {
     "ascli": "~1",
-    "bytebuffer": "~4 >=4.1",
-    "glob": "^5.0.10",
+    "bytebuffer": "~5",
+    "glob": "^7.0.5",
     "yargs": "^3.10.0"
   },
   "devDependencies": {
@@ -55,11 +55,11 @@
     "build": "node scripts/build.js",
     "descriptor2json": "node bin/pbjs src/google/protobuf/descriptor.proto --target=json > src/google/protobuf/descriptor.json",
     "compile": "npm run-script compile-full && npm run-script compile-light",
-    "compile-full": "ccjs dist/ProtoBuf.js --create_source_map=dist/ProtoBuf.min.map --compilation_level=SIMPLE_OPTIMIZATIONS > dist/ProtoBuf.min.js",
-    "compile-light": "ccjs dist/ProtoBuf-light.js --create_source_map=dist/ProtoBuf-light.min.map --compilation_level=SIMPLE_OPTIMIZATIONS > dist/ProtoBuf-light.min.js",
+    "compile-full": "ccjs dist/protobuf.js --create_source_map=dist/protobuf.min.map --compilation_level=SIMPLE_OPTIMIZATIONS > dist/protobuf.min.js",
+    "compile-light": "ccjs dist/protobuf-light.js --create_source_map=dist/protobuf-light.min.map --compilation_level=SIMPLE_OPTIMIZATIONS > dist/protobuf-light.min.js",
     "compress": "npm run-script compress-full && npm run-script compress-light",
-    "compress-full": "gzip -c -9 dist/ProtoBuf.min.js > dist/ProtoBuf.min.js.gz",
-    "compress-light": "gzip -c -9 dist/ProtoBuf-light.min.js > dist/ProtoBuf-light.min.js.gz",
+    "compress-full": "gzip -c -9 dist/protobuf.min.js > dist/protobuf.min.js.gz",
+    "compress-light": "gzip -c -9 dist/protobuf-light.min.js > dist/protobuf-light.min.js.gz",
     "jsdoc": "node node_modules/jsdoc/jsdoc.js -c jsdoc.json"
   }
 }

package/scripts/build.js

@@ -1,58 +1,53 @@
-/*
- Copyright 2013 Daniel Wirtz <dcode@dcode.io>
-
- Licensed under the Apache License, Version 2.0 (the "License");
- you may not use this file except in compliance with the License.
- You may obtain a copy of the License at
-
- http://www.apache.org/licenses/LICENSE-2.0
-
- Unless required by applicable law or agreed to in writing, software
- distributed under the License is distributed on an "AS IS" BASIS,
- WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
- See the License for the specific language governing permissions and
- limitations under the License.
- */
-
-/**
- * @overview ProtoBuf.js Build Script (c) 2014 Daniel Wirtz <dcode@dcode.io>
- * Released under the Apache License, Version 2.0
- * see: https://github.com/dcodeIO/ProtoBuf.js for details
- */
-var MetaScript = require("metascript"),
-    path = require("path"),
-    fs = require("fs");
-
-var rootDir = path.join(__dirname, ".."),
-    srcDir  = path.join(rootDir, "src"),
-    distDir = path.join(rootDir, "dist"),
-    pkg = require(path.join(rootDir, "package.json")),
-    filename;
-
-var scope = {
-    VERSION: pkg.version,           // Version
-    DOTPROTO: true                  // Whether to include the ProtoBuf.DotProto package for .proto syntax support
-};
-
-// Make full build
-console.log("Building ProtoBuf.js with scope", JSON.stringify(scope, null, 2));
-fs.writeFileSync(
-    path.join(distDir, "ProtoBuf.js"),
-    MetaScript.transform(fs.readFileSync(filename = path.join(srcDir, "wrap.js")), filename, scope, srcDir)
-);
-
-// Make light build
-scope.DOTPROTO = false;
-console.log("Building ProtoBuf-light.js with scope", JSON.stringify(scope, null, 2));
-fs.writeFileSync(
-    path.join(distDir, "ProtoBuf-light.js"),
-    MetaScript.transform(fs.readFileSync(filename = path.join(srcDir, "wrap.js")), filename, scope, srcDir)
-);
-
-// Update bower.json
-scope = { VERSION: pkg.version };
-console.log("Updating bower.json with scope", JSON.stringify(scope, null, 2));
-fs.writeFileSync(
-    path.join(rootDir, "bower.json"),
-    MetaScript.transform(fs.readFileSync(filename = path.join(srcDir, "bower.json")), filename, scope, srcDir)
-);
+/*
+ Copyright 2013 Daniel Wirtz <dcode@dcode.io>
+
+ Licensed under the Apache License, Version 2.0 (the "License");
+ you may not use this file except in compliance with the License.
+ You may obtain a copy of the License at
+
+ http://www.apache.org/licenses/LICENSE-2.0
+
+ Unless required by applicable law or agreed to in writing, software
+ distributed under the License is distributed on an "AS IS" BASIS,
+ WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ See the License for the specific language governing permissions and
+ limitations under the License.
+ */
+
+var MetaScript = require("metascript"),
+    path = require("path"),
+    fs = require("fs");
+
+var rootDir = path.join(__dirname, ".."),
+    srcDir  = path.join(rootDir, "src"),
+    distDir = path.join(rootDir, "dist"),
+    pkg = require(path.join(rootDir, "package.json")),
+    filename;
+
+var scope = {
+    VERSION: pkg.version,           // Version
+    DOTPROTO: true                  // Whether to include the ProtoBuf.DotProto package for .proto syntax support
+};
+
+// Make full build
+console.log("Building protobuf.js with scope", JSON.stringify(scope, null, 2));
+fs.writeFileSync(
+    path.join(distDir, "protobuf.js"),
+    MetaScript.transform(fs.readFileSync(filename = path.join(srcDir, "wrap.js")), filename, scope, srcDir)
+);
+
+// Make light build
+scope.DOTPROTO = false;
+console.log("Building protobuf-light.js with scope", JSON.stringify(scope, null, 2));
+fs.writeFileSync(
+    path.join(distDir, "protobuf-light.js"),
+    MetaScript.transform(fs.readFileSync(filename = path.join(srcDir, "wrap.js")), filename, scope, srcDir)
+);
+
+// Update bower.json
+scope = { VERSION: pkg.version };
+console.log("Updating bower.json with scope", JSON.stringify(scope, null, 2));
+fs.writeFileSync(
+    path.join(rootDir, "bower.json"),
+    MetaScript.transform(fs.readFileSync(filename = path.join(srcDir, "bower.json")), filename, scope, srcDir)
+);

package/src/ProtoBuf/Builder/Message.js

@@ -110,9 +110,11 @@ MessagePrototype.$add = MessagePrototype.add;
 MessagePrototype.set = function(keyOrObj, value, noAssert) {
     if (keyOrObj && typeof keyOrObj === 'object') {
         noAssert = value;
-        for (var ikey in keyOrObj)
-            if (keyOrObj.hasOwnProperty(ikey) && typeof (value = keyOrObj[ikey]) !== 'undefined')
+        for (var ikey in keyOrObj) {
+            // Check if virtual oneof field - don't set these
+            if (keyOrObj.hasOwnProperty(ikey) && typeof (value = keyOrObj[ikey]) !== 'undefined' && T._oneofsByName[ikey] === undefined)
                 this.$set(ikey, value, noAssert);
+        }
         return this;
     }
     var field = T._fieldsByName[keyOrObj];
@@ -340,18 +342,19 @@ MessagePrototype.calculate = function() {
  * @name ProtoBuf.Builder.Message#encodeDelimited
  * @function
  * @param {(!ByteBuffer|boolean)=} buffer ByteBuffer to encode to. Will create a new one and flip it if omitted.
+ * @param {boolean=} noVerify Whether to not verify field values, defaults to `false`
  * @return {!ByteBuffer} Encoded message as a ByteBuffer
  * @throws {Error} If the message cannot be encoded or if required fields are missing. The later still
  *  returns the encoded ByteBuffer in the `encoded` property on the error.
  * @expose
  */
-MessagePrototype.encodeDelimited = function(buffer) {
+MessagePrototype.encodeDelimited = function(buffer, noVerify) {
     var isNew = false;
     if (!buffer)
         buffer = new ByteBuffer(),
         isNew = true;
     var enc = new ByteBuffer().LE();
-    T.encode(this, enc).flip();
+    T.encode(this, enc, noVerify).flip();
     buffer.writeVarint32(enc.remaining());
     buffer.append(enc);
     return isNew ? buffer.flip() : buffer;
@@ -498,7 +501,7 @@ function cloneRaw(obj, binaryAsBase64, longsAsStrings, resolvedType) {
         return binaryAsBase64 ? obj.toBase64() : obj.toBuffer();
     // Convert Longs to proper objects or strings
     if (ProtoBuf.Long.isLong(obj))
-        return longsAsStrings ? obj.toString() : new ProtoBuf.Long(obj);
+        return longsAsStrings ? obj.toString() : ProtoBuf.Long.fromValue(obj);
     var clone;
     // Clone arrays
     if (Array.isArray(obj)) {
@@ -560,6 +563,7 @@ MessagePrototype.encodeJSON = function() {
  * @name ProtoBuf.Builder.Message.decode
  * @function
  * @param {!ByteBuffer|!ArrayBuffer|!Buffer|string} buffer Buffer to decode from
+ * @param {(number|string)=} length Message length. Defaults to decode all the remainig data.
  * @param {string=} enc Encoding if buffer is a string: hex, utf8 (not recommended), defaults to base64
  * @return {!ProtoBuf.Builder.Message} Decoded message
  * @throws {Error} If the message cannot be decoded or if required fields are missing. The later still
@@ -568,13 +572,17 @@ MessagePrototype.encodeJSON = function() {
  * @see ProtoBuf.Builder.Message.decode64
  * @see ProtoBuf.Builder.Message.decodeHex
  */
-Message.decode = function(buffer, enc) {
+Message.decode = function(buffer, length, enc) {
+    if (typeof length === 'string')
+        enc = length,
+        length = -1;
     if (typeof buffer === 'string')
         buffer = ByteBuffer.wrap(buffer, enc ? enc : "base64");
-    buffer = ByteBuffer.isByteBuffer(buffer) ? buffer : ByteBuffer.wrap(buffer); // May throw
+    else if (!ByteBuffer.isByteBuffer(buffer))
+        buffer = ByteBuffer.wrap(buffer); // May throw
     var le = buffer.littleEndian;
     try {
-        var msg = T.decode(buffer.LE());
+        var msg = T.decode(buffer.LE(), length);
         buffer.LE(le);
         return msg;
     } catch (e) {
@@ -597,7 +605,8 @@ Message.decode = function(buffer, enc) {
 Message.decodeDelimited = function(buffer, enc) {
     if (typeof buffer === 'string')
         buffer = ByteBuffer.wrap(buffer, enc ? enc : "base64");
-    buffer = ByteBuffer.isByteBuffer(buffer) ? buffer : ByteBuffer.wrap(buffer); // May throw
+    else if (!ByteBuffer.isByteBuffer(buffer))
+        buffer = ByteBuffer.wrap(buffer); // May throw
     if (buffer.remaining() < 1)
         return null;
     var off = buffer.offset,

package/src/ProtoBuf/Builder/Service.js

@@ -77,6 +77,9 @@ for (var i=0; i<rpc.length; i++) {
                         callback(err);
                         return;
                     }
+                    // Coalesce to empty string when service response has empty content
+                    if (res === null)
+                        res = ''
                     try { res = method.resolvedResponseType.clazz.decode(res); } catch (notABuffer) {}
                     if (!res || !(res instanceof method.resolvedResponseType.clazz)) {
                         callback(Error("Illegal response type received in service method "+ T.name+"#"+method.name));

package/src/ProtoBuf/Builder.js

@@ -243,13 +243,12 @@ ProtoBuf.Builder = (function(ProtoBuf, Lang, Reflect) {
                             subObj.push(svc);
                         });
 
-                    // Set extension range
+                    // Set extension ranges
                     if (def["extensions"]) {
-                        obj.extensions = def["extensions"];
-                        if (obj.extensions[0] < ProtoBuf.ID_MIN)
-                            obj.extensions[0] = ProtoBuf.ID_MIN;
-                        if (obj.extensions[1] > ProtoBuf.ID_MAX)
-                            obj.extensions[1] = ProtoBuf.ID_MAX;
+                        if (typeof def["extensions"][0] === 'number') // pre 5.0.1
+                            obj.extensions = [ def["extensions"] ];
+                        else
+                            obj.extensions = def["extensions"];
                     }
 
                     // Create on top of current namespace
@@ -288,8 +287,16 @@ ProtoBuf.Builder = (function(ProtoBuf, Lang, Reflect) {
                         def["fields"].forEach(function(fld) {
                             if (obj.getChild(fld['id']|0) !== null)
                                 throw Error("duplicate extended field id in "+obj.name+": "+fld['id']);
-                            if (fld['id'] < obj.extensions[0] || fld['id'] > obj.extensions[1])
-                                throw Error("illegal extended field id in "+obj.name+": "+fld['id']+" ("+obj.extensions.join(' to ')+" expected)");
+                            // Check if field id is allowed to be extended
+                            if (obj.extensions) {
+                                var valid = false;
+                                obj.extensions.forEach(function(range) {
+                                    if (fld["id"] >= range[0] && fld["id"] <= range[1])
+                                        valid = true;
+                                });
+                                if (!valid)
+                                    throw Error("illegal extended field id in "+obj.name+": "+fld['id']+" (not within valid ranges)");
+                            }
                             // Convert extension field names to camel case notation if the override is set
                             var name = fld["name"];
                             if (this.options['convertFieldsToCamelCase'])
@@ -370,7 +377,11 @@ ProtoBuf.Builder = (function(ProtoBuf, Lang, Reflect) {
                 root = require("path")['resolve'](root);
             if (root.indexOf("\\") >= 0 || filename.file.indexOf("\\") >= 0)
                 delim = '\\';
-            var fname = root + delim + filename.file;
+            var fname;
+            if (ProtoBuf.Util.IS_NODE)
+                fname = require("path")['join'](root, filename.file);
+            else
+                fname = root + delim + filename.file;
             if (this.files[fname] === true)
                 return this.reset();
             this.files[fname] = true;
@@ -416,7 +427,10 @@ ProtoBuf.Builder = (function(ProtoBuf, Lang, Reflect) {
                     var importFilename = json['imports'][i];
                     if (importFilename === "google/protobuf/descriptor.proto")
                         continue; // Not needed and therefore not used
-                    importFilename = importRoot + delim + importFilename;
+                    if (ProtoBuf.Util.IS_NODE)
+                        importFilename = require("path")['join'](importRoot, importFilename);
+                    else
+                        importFilename = importRoot + delim + importFilename;
                     if (this.files[importFilename] === true)
                         continue; // Already imported
                     if (/\.proto$/i.test(importFilename) && !ProtoBuf.DotProto)       // If this is a light build
@@ -514,6 +528,16 @@ ProtoBuf.Builder = (function(ProtoBuf, Lang, Reflect) {
                 this.ptr.keyType = ProtoBuf.TYPES[this.ptr.keyType];
             }
 
+            // If it's a repeated and packable field then proto3 mandates it should be packed by
+            // default
+            if (
+              this.ptr.syntax === 'proto3' &&
+              this.ptr.repeated && this.ptr.options.packed === undefined &&
+              ProtoBuf.PACKABLE_WIRE_TYPES.indexOf(this.ptr.type.wireType) !== -1
+            ) {
+              this.ptr.options.packed = true;
+            }
+
         } else if (this.ptr instanceof ProtoBuf.Reflect.Service.Method) {
 
             if (this.ptr instanceof ProtoBuf.Reflect.Service.RPCMethod) {

package/src/ProtoBuf/DotProto/Parser.js

@@ -50,7 +50,8 @@ ParserPrototype.parse = function() {
         // "syntax": undefined
     };
     var token,
-        head = true;
+        head = true,
+        weak;
     try {
         while (token = this.tn.next()) {
             switch (token) {
@@ -67,11 +68,12 @@ ParserPrototype.parse = function() {
                     if (!head)
                         throw Error("unexpected 'import'");
                     token = this.tn.peek();
-                    if (token === "public") // ignored
+                    if (token === "public" || (weak = token === "weak")) // token ignored
                         this.tn.next();
                     token = this._readString();
                     this.tn.skip(";");
-                    topLevel["imports"].push(token);
+                    if (!weak) // import ignored
+                        topLevel["imports"].push(token);
                     break;
                 case 'syntax':
                     if (!head)
@@ -396,6 +398,7 @@ ParserPrototype._parseMessage = function(parent, fld) {
         "enums": [],
         "messages": [],
         "options": {},
+        "services": [],
         "oneofs": {}
         // "extensions": undefined
     };
@@ -422,8 +425,16 @@ ParserPrototype._parseMessage = function(parent, fld) {
             this._parseMessage(msg);
         else if (token === "option")
             this._parseOption(msg);
+        else if (token === "service")
+            this._parseService(msg);
         else if (token === "extensions")
-            this._parseExtensions(msg);
+            if (msg.hasOwnProperty("extensions")) {
+                msg["extensions"] = msg["extensions"].concat(this._parseExtensionRanges())
+            } else {
+                msg["extensions"] = this._parseExtensionRanges();
+            }
+        else if (token === "reserved")
+            this._parseIgnored(); // TODO
         else if (token === "extend")
             this._parseExtend(msg);
         else if (Lang.TYPEREF.test(token)) {
@@ -438,6 +449,16 @@ ParserPrototype._parseMessage = function(parent, fld) {
     return msg;
 };
 
+/**
+ * Parses an ignored statement.
+ * @private
+ */
+ParserPrototype._parseIgnored = function() {
+    while (this.tn.peek() !== ';')
+        this.tn.next();
+    this.tn.skip(";");
+};
+
 /**
  * Parses a message field.
  * @param {!Object} msg Message definition
@@ -600,29 +621,43 @@ ParserPrototype._parseEnum = function(msg) {
 };
 
 /**
- * Parses an extensions statement.
- * @param {!Object} msg Message object
+ * Parses extension / reserved ranges.
+ * @returns {!Array.<!Array.<number>>}
  * @private
  */
-ParserPrototype._parseExtensions = function(msg) {
-    var token = this.tn.next(),
+ParserPrototype._parseExtensionRanges = function() {
+    var ranges = [];
+    var token,
+        range,
+        value;
+    do {
         range = [];
-    if (token === "min")
-        range.push(ProtoBuf.ID_MIN);
-    else if (token === "max")
-        range.push(ProtoBuf.ID_MAX);
-    else
-        range.push(mkNumber(token));
-    this.tn.skip("to");
-    token = this.tn.next();
-    if (token === "min")
-        range.push(ProtoBuf.ID_MIN);
-    else if (token === "max")
-        range.push(ProtoBuf.ID_MAX);
-    else
-        range.push(mkNumber(token));
+        while (true) {
+            token = this.tn.next();
+            switch (token) {
+                case "min":
+                    value = ProtoBuf.ID_MIN;
+                    break;
+                case "max":
+                    value = ProtoBuf.ID_MAX;
+                    break;
+                default:
+                    value = mkNumber(token);
+                    break;
+            }
+            range.push(value);
+            if (range.length === 2)
+                break;
+            if (this.tn.peek() !== "to") {
+                range.push(value);
+                break;
+            }
+            this.tn.next();
+        }
+        ranges.push(range);
+    } while (this.tn.omit(","));
     this.tn.skip(";");
-    msg["extensions"] = range;
+    return ranges;
 };
 
 /**

package/src/ProtoBuf/Reflect/Element.js

@@ -15,9 +15,11 @@
  * converted to string form if so.
  * @param {string} syntax Syntax level of defining message type, e.g.,
  * proto2 or proto3.
+ * @param {string} name Name of the field containing this element (for error
+ * messages)
  * @constructor
  */
-var Element = function(type, resolvedType, isMapKey, syntax) {
+var Element = function(type, resolvedType, isMapKey, syntax, name) {
 
     /**
      * Element type, as a string (e.g., int32).
@@ -43,6 +45,12 @@ var Element = function(type, resolvedType, isMapKey, syntax) {
      */
     this.syntax = syntax;
 
+    /**
+     * Name of the field containing this element (for error messages)
+     * @type {string}
+     */
+    this.name = name;
+
     if (isMapKey && ProtoBuf.MAP_KEY_TYPES.indexOf(type) < 0)
         throw Error("Invalid map key type: " + type.name);
 };
@@ -93,6 +101,10 @@ function mkLong(value, unsigned) {
     throw Error("not convertible to Long");
 }
 
+ElementPrototype.toString = function() {
+    return (this.name || '') + (this.isMapKey ? 'map' : 'value') + ' element';
+}
+
 /**
  * Checks if the given value can be set for an element of this type (singular
  * field or one element of a repeated field or map).
@@ -102,9 +114,10 @@ function mkLong(value, unsigned) {
  * @expose
  */
 ElementPrototype.verifyValue = function(value) {
-    var fail = function(val, msg) {
-        throw Error("Illegal value for "+this.toString(true)+" of type "+this.type.name+": "+val+" ("+msg+")");
-    }.bind(this);
+    var self = this;
+    function fail(val, msg) {
+        throw Error("Illegal value for "+self.toString(true)+" of type "+self.type.name+": "+val+" ("+msg+")");
+    }
     switch (this.type) {
         // Signed 32bit
         case ProtoBuf.TYPES["int32"]:

package/src/ProtoBuf/Reflect/Message/Field.js

@@ -141,9 +141,9 @@ var FieldPrototype = Field.prototype = Object.create(T.prototype);
  * @expose
  */
 FieldPrototype.build = function() {
-    this.element = new Element(this.type, this.resolvedType, false, this.syntax);
+    this.element = new Element(this.type, this.resolvedType, false, this.syntax, this.name);
     if (this.map)
-        this.keyElement = new Element(this.keyType, undefined, true, this.syntax);
+        this.keyElement = new Element(this.keyType, undefined, true, this.syntax, this.name);
 
     // In proto3, fields do not have field presence, and every field is set to
     // its type's default value ("", 0, 0.0, or false).
@@ -165,9 +165,10 @@ FieldPrototype.build = function() {
  */
 FieldPrototype.verifyValue = function(value, skipRepeated) {
     skipRepeated = skipRepeated || false;
-    var fail = function(val, msg) {
-        throw Error("Illegal value for "+this.toString(true)+" of type "+this.type.name+": "+val+" ("+msg+")");
-    }.bind(this);
+    var self = this;
+    function fail(val, msg) {
+        throw Error("Illegal value for "+self.toString(true)+" of type "+self.type.name+": "+val+" ("+msg+")");
+    }
     if (value === null) { // NULL values for optional fields
         if (this.required)
             fail(typeof value, "required");

package/src/ProtoBuf/Reflect/Message.js

@@ -20,10 +20,10 @@ var Message = function(builder, parent, name, options, isGroup, syntax) {
 
     /**
      * Extensions range.
-     * @type {!Array.<number>}
+     * @type {!Array.<number>|undefined}
      * @expose
      */
-    this.extensions = [ProtoBuf.ID_MIN, ProtoBuf.ID_MAX];
+    this.extensions = undefined;
 
     /**
      * Runtime message class.
@@ -94,6 +94,7 @@ MessagePrototype.build = function(rebuild) {
     this._fields = [];
     this._fieldsById = {};
     this._fieldsByName = {};
+    this._oneofsByName = {};
     for (var i=0, k=this.children.length, child; i<k; i++) {
         child = this.children[i];
         if (child instanceof Enum || child instanceof Message || child instanceof Service) {
@@ -105,6 +106,9 @@ MessagePrototype.build = function(rebuild) {
             this._fields.push(child),
             this._fieldsById[child.id] = child,
             this._fieldsByName[child.name] = child;
+        else if (child instanceof Message.OneOf) {
+            this._oneofsByName[child.name] = child;
+        }
         else if (!(child instanceof Message.OneOf) && !(child instanceof Extension)) // Not built
             throw Error("Illegal reflect child of "+this.toString(true)+": "+this.children[i].toString(true));
     }
@@ -204,14 +208,15 @@ function skipTillGroupEnd(expectedId, buf) {
 /**
  * Decodes an encoded message and returns the decoded message.
  * @param {ByteBuffer} buffer ByteBuffer to decode from
- * @param {number=} length Message length. Defaults to decode all the available data.
+ * @param {number=} length Message length. Defaults to decode all remaining data.
  * @param {number=} expectedGroupEndId Expected GROUPEND id if this is a legacy group
  * @return {ProtoBuf.Builder.Message} Decoded message
  * @throws {Error} If the message cannot be decoded
  * @expose
  */
 MessagePrototype.decode = function(buffer, length, expectedGroupEndId) {
-    length = typeof length === 'number' ? length : -1;
+    if (typeof length !== 'number')
+        length = -1;
     var start = buffer.offset,
         msg = new (this.clazz)(),
         tag, wireType, id, field;