Do not skip optional delimiter after aggregate options, fixes #520

dcodeIO · dcodeIO · commit 38d867fc50a4 · 2016-12-07T10:55:16.000+01:00
diff --git a/src/parse.js b/src/parse.js
@@ -393,7 +393,6 @@ function parse(source, root) {
                 else
                     parseOptionValue(parent, name);
             }
-            skip(s_semi, true);
         } else
             setOption(parent, name, readValue(true));
         // Does not enforce a delimiter to be universal
diff --git a/tests/data/service.proto b/tests/data/service.proto
@@ -0,0 +1,16 @@
+syntax = "proto3";
+package myservice;
+
+message DoSomethingRequest {
+}
+
+message DoSomethingResponse {
+}
+
+service MyService {
+  rpc DoSomething(DoSomethingRequest) returns (DoSomethingResponse) {
+    option (google.api.http) = {
+      get: "/v1/dosomething"
+    };
+  };
+}

Original file line number	Diff line number	Diff line change
`@@ -393,7 +393,6 @@ function parse(source, root) {`
`393`	`393`	`else`
`394`	`394`	`parseOptionValue(parent, name);`
`395`	`395`	`}`
`396`		`- skip(s_semi, true);`
`397`	`396`	`} else`
`398`	`397`	`setOption(parent, name, readValue(true));`
`399`	`398`	`// Does not enforce a delimiter to be universal`