yoheimuta / go-protoparser

Yet another Go package which parses a Protocol Buffer file (proto2+proto3)
MIT License
179 stars 20 forks source link

Improve flexibility of parsing tool #43

Closed PaulSonOfLars closed 4 years ago

PaulSonOfLars commented 4 years ago

The parser reports a lot of protos as broken due to protoc being more flexible and forgiving than the official spec suggests. This PR adds a few of these edge cases, such as:

Let me know if any of these require extra data. I will be adding tests for these shortly.

PaulSonOfLars commented 4 years ago

This addresses the following issues: https://github.com/yoheimuta/protolint/issues/108 https://github.com/yoheimuta/protolint/issues/109

Tested locally on various problematic protos (thank god for gomodules!)