-
-
Notifications
You must be signed in to change notification settings - Fork 16
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[Bug report] Parsing error when the message name is contained in the message fields #39
Comments
@g-berthiaume Use like: it will fix:
|
hmm, yep. the word But I think this word "type" is a very common word, so I decided to solve this. A newer version of bitproto, v0.4.2 was just uploaded, which fixed this problem. Now the following message parses well:
You can upgrade bitproto via |
Interesting ! As I'm starting to work on a basic compiler, I can appreciate the complexity of those kinds of bugs. On another note: Is it possible that the package builder executes |
@g-berthiaume Thanks for reporting this bug.
Or you can download the binary directly from https://github.com/hit9/bitproto/releases/tag/v0.4.4 |
I think the parser will fail if the message name is contained in the message field type.
Steps to reproduce
Using the following schema:
When compiling
My guess is that this error is due to a kind of name collision in the lexer.
The text was updated successfully, but these errors were encountered: