-
Notifications
You must be signed in to change notification settings - Fork 610
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 Doxygen warnings for C++ code #6608
Conversation
78a11e3
to
9421382
Compare
I think Doxygen is failing to parse requires clauses. Here's how I worked around them in Sleipnir: // FIXME: Doxygen gives internal inconsistency errors:
// scope for class sleipnir::fmt::formatter< Derived, CharT > not found!
// scope for class sleipnir::fmt::formatter< sleipnir::Variable > not found!
// scope for class sleipnir::fmt::formatter< T > not found!
//! @cond Doxygen_Suppress
Badly parsed code here
//! @endcond |
It looks like the latest Doxygen (1.10.0) has a fix for parsing requires |
If so that would probably be a good reason to upgrade. Might also fix the parsing bugs for other parts. I'll have a go fixing the warning around WPIWarnings.mac and WPIErrors.mac without a supression and wait for the WPILib mirror of Doxygen releases to have latest Doxygen before trying to fix everything else. |
5cc570f
to
df49e1a
Compare
bd47119
to
45a48c1
Compare
Resolves #6221