-
Notifications
You must be signed in to change notification settings - Fork 10
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
Param names #94
Param names #94
Conversation
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.
Couple of minor nitpicks, but I can go either way. Otherwise I think it's good to go notwithstanding what we discussed in the draft version.
It's also a great clean-up of course, congrats.
We'll want to ensure the array dimensions are handled in the correct order in upcoming changes. Add a test for it.
Pass cursor to _decl_fixup() as we'll need it in upcoming changes.
Combine both array and function pointer fixups into one. Based on the work by Robin Bourianes <robin.bourianes@kalisio.com> [1]. [1] #86
[Jani: whitespace changes to reflect code changes.]
[Jani: whitespace changes to reflect code changes.]
Fix output for functions without parameters.
Better describe the cases for no parameters and empty parameter list.
Thanks for the review! Fixed the code change, left the test as-is, and merged. |
Updated version of #93.
I'm still not entirely happy about how all the spacing happens, but I think it's passable. Fixed void functions while at it, and added some more tests.