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
Prepare headers for rz-bindgen #2786
Conversation
5e04383
to
989ae21
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is quite cool!
Where is the code you used to do this? Do you plan to add anything else to this PR?
I'm adding type annotations whenever my binding generator needs a container type's contents, so it only covers parts that are being bound. I'm working on adding more bindings here |
What do you think if you make this PR "ready" and we start merging the current things? I'd like to avoid huge PRs. |
659c7d5
to
af5d3db
Compare
@wingdeans please resolve the conflict and we will merge this. |
* add some missing /*<types>*/ to rz_core, bin, analysis, cons * add annotation attributes for rz-bindgen * fix extern "C" in some headers * rename RzBuffer functions from length to len * fix rz_core_analysis_name naming conflict
Checklist
Detailed description
Container types (eg. RzList) in headers should specify the type of their contents in a block comment.
The format
/*<content_type>*/
should work. I've also seen/*<type_key, type_value>*/
for hashtables...
Test plan
I am working on scripts to warn about missing annotations.
There isn't a way to verify correctness of annotations (yet).