address comments

Author: sofisl

src/parse.js

@@ -372,15 +372,13 @@ function parse(source, root, options) {
                 case "required":
                     if (edition)
                         throw illegal(token);
-                    break;
+                /* eslint-disable no-fallthrough */
                 case "repeated":
                     parseField(type, token);
                     break;
 
                 case "optional":
                     /* istanbul ignore if */
-                    // console.log('in optional?')
-                    // console.log('')
                     if (isProto3) {
                         parseField(type, "proto3_optional");
                     } else if (edition) {
@@ -404,8 +402,9 @@ function parse(source, root, options) {
 
                 default:
                     /* istanbul ignore if */
-                    if (!isProto3 && !edition || !typeRefRe.test(token))
+                    if (!isProto3 && !edition || !typeRefRe.test(token)) {
                         throw illegal(token);
+                    }
 
                     push(token);
                     parseField(type, "optional");

src/util.js

@@ -171,10 +171,10 @@ util.decorateEnum = function decorateEnum(object) {
  * @param {Object.<string,*>} dst Destination object
  * @param {string} path dot '.' delimited path of the property to set
  * @param {Object} value the value to set
- * @param {boolean} overWrite whether or not to concatenate the values into an array or overwrite
+ * @param {boolean} overWrite whether or not to concatenate the values into an array or overwrite; defaults to false.
  * @returns {Object.<string,*>} Destination object
  */
-util.setProperty = function setProperty(dst, path, value, overWrite) {
+util.setProperty = function setProperty(dst, path, value, overWrite = false) {
     function setProp(dst, path, value) {
         var part = path.shift();
         if (part === "__proto__" || part === "prototype") {