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
Emit GetRootAs methods for all types in Go and Python #363
Conversation
the "GetRootAs" methods are really convenient and necessary for reading messages consisting only non-root_type object. |
Looks good to me, @rw ? |
b611dc4
to
fe2f8d3
Compare
Thanks for submitting this PR. Please add tests that verify all of these work correctly for the example Monster (and so that we can prevent regressions). Otherwise LGTM. |
Bump @danring |
@rw Apologies, I was offline for a week. I'll add tests soon. |
@rw Are the added tests sufficient? |
Bump @rw -- Any further tests needed? |
Hi! Any chance to have this merged somehow? Would help to use gRPC+flatbuffers in Go. Thanks! |
@rw? |
status? |
Thank you, merged! |
Handles part of #341, adding
GetRootAs*
methods to non-root tables. Basically just applies the same change from 9f506f5 toidl_gen_python.cpp
andidl_gen_go.cpp
. (Also removes an extra newline from generated Python.)