You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Welp, I thought I had implemented support for this correctly. I see the intent to support this in the grammar.
But I think there is a legit shift-reduce conflict in the grammar; there are actually many such conflicts reported by yacc, but I thought they were all harmless and that the generated parser would still work. In this case, I think the conflict causes the parser to not properly recognize this production. Luckily, I think it's a pretty easy fix.
jhump
changed the title
Custom options parting issue from protocolbuffers/protobuf unit test file
protoparse: trying to set extension field on a custom option whose type is message results in syntax error
Apr 11, 2020
https://github.com/protocolbuffers/protobuf/blob/master/src/google/protobuf/unittest_custom_options.proto#L281
It seems that the double parentheses are tripping up protoreflect
The text was updated successfully, but these errors were encountered: