Complex options cannot be parsed about proto HOT 7 CLOSED

There is actually an alternate syntax without the ':' after 'foo' that is also valid in Protobuf:

syntax = "proto3";

package baz;

option (foo.bar) = {
  woot: 100
  foo {
    hello: 200
  }
};

See https://github.com/googleapis/googleapis/blob/master/google/privacy/dlp/v2beta2/dlp.proto#L96

Here's a diff on service_test.go that breaks:

$ git diff
diff --git a/service_test.go b/service_test.go
index 058c885..0a4fc5d 100644
--- a/service_test.go
+++ b/service_test.go
@@ -86,6 +86,10 @@ func TestRPCWithOptionAggregateSyntax(t *testing.T) {
                        option (test_ident) = {
                                test: "test"
                                test2:"test2"
+                               addtional_bindings {
+                                       hello: "value"
+                                       hello2: "value2"
+                               }
                        }; // inline test_ident
                }
        }`

This results in found "{" but expected [option aggregate key colon :]

from proto.

ATM aggregations are stored in AggregatedConstants. With this recursive example, this is no longer valid. Need to think about this a bit more

from proto.

in the PR, Issue #60 , the constants are flattened.
in your example given, there will be a constant called foo.hello

from proto.

This won't work for other arbitrary cases. For example, what if I have a repeated field in the message? so foo.hello is a repeated string? What if it has a oneof, etc?

This might be fine to start, but just of note :-)

from proto.

This is still broken, will write a test.

from proto.

See the PR I just wrote, two things need to be handled:

1. Having a : after the key before a complex option
2. Having the value be on a separate line

from proto.

closing this as #60 is merged

from proto.