fix: allow file-level options everywhere in the file

JustinBeckwith · web-flow · commit b70eebd86e11 · 2019-04-18T21:31:43.000-07:00
allow file-level options everywhere in the file
diff --git a/src/parse.js b/src/parse.js
@@ -714,10 +714,6 @@ function parse(source, root, options) {
 
             case "option":
 
-                /* istanbul ignore if */
-                if (!head)
-                    throw illegal(token);
-
                 parseOption(ptr, token);
                 skip(";");
                 break;
diff --git a/tests/data/test.proto b/tests/data/test.proto
@@ -33,7 +33,6 @@
 syntax = "proto2";
 
 option java_package = "com.google.apps.jspb.proto";
-option java_multiple_files = true;
 
 import "google/protobuf/descriptor.proto";
 
@@ -267,4 +266,6 @@ message Deeply {
       optional int32 count = 1;
     }
   }
-}
+}
+
+option java_multiple_files = true;

Original file line number	Diff line number	Diff line change
`@@ -33,7 +33,6 @@`
`33`	`33`	`syntax = "proto2";`
`34`	`34`
`35`	`35`	`option java_package = "com.google.apps.jspb.proto";`
`36`		`-option java_multiple_files = true;`
`37`	`36`
`38`	`37`	`import "google/protobuf/descriptor.proto";`
`39`	`38`
`@@ -267,4 +266,6 @@ message Deeply {`
`267`	`266`	`optional int32 count = 1;`
`268`	`267`	`}`
`269`	`268`	`}`
`270`		`-}`
	`269`	`+}`
	`270`	`+`
	`271`	`+option java_multiple_files = true;`