-
-
Notifications
You must be signed in to change notification settings - Fork 349
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
Fix member groupings in doxygen #1276
Conversation
79de571
to
acbd4a5
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.
Thanks for working through this, @ischoegl. I know this kind of comment reformatting can be a bit tedious. I had only a couple spots where I think there were some additional things worth cleaning up.
acbd4a5
to
eb64564
Compare
@speth ... done. It was tedious, but searching for |
Ended up taking care of module groupings also, so every grouping is now formatted consistently |
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.
Thanks, @ischoegl. Hopefully, having the existing style for these be consistent will encourage consistent usage in the future as well.
Changes proposed in this pull request
Per #1275, there are two working formatting variants of member grouping commonly found (while others don't work and don't produce the expected output). This PR consistently applies the following
If applicable, fill in the issue number this pull request is fixing
Closes #1275
If applicable, provide an example illustrating new features this pull request is introducing
Checklist
scons build
&scons test
) and unit tests address code coverage