Opened 7 years ago
Closed 6 years ago
#13107 closed Patch - Bug Fix (fixed)
Clean up almost all doxygen warnings.
Reported by: | Owned by: | Stuart Auchterlonie | |
---|---|---|---|
Priority: | minor | Milestone: | 30.0 |
Component: | MythTV - General | Version: | Master Head |
Severity: | medium | Keywords: | |
Cc: | Ticket locked: | no |
Description
1) Update the base configuration file to doxygen 1.8.13.
2) Tweak the configuration file to:
a) Use the LOOKUP_CACHE_SIZE recommended by doxygen. b) Fix up the exclude paths patterns. c) Increase the number of nodes in a graph, and use the default system font.
3) Update the main architecture document to eliminate warnings. Add a couple of new groups for the libraries for later documentation. Remove some unintentional links. Remove the Transcoding Daemon documentation.
4) Fix doxygen complaints about macros before the keyword 'public'. Doxygen gets easily confused, and unless the token before 'public' is a semicolon, it doesn't recognize the keyword
5) Fix doxygen warning in all the source files.
a) Function parameter names must be the same across function declaration, function declaration, and doxygen comment declaration. If the function is inherited from a parent class and has no documentation of its own, then the names must also match those of the parent class function.
b) Remove parameter documentation if parameters have been removed.
c) Add parameter documentation if parameters have been added.
d) Fix punctuation in comments that causes a doxygen warning, or makes doxygen think that a link should be created.
e) Fix doxygen commands to be acceptable (wrong case, wrong command, etc). Convert parameter arguments from javadoc style (@my_var) to doxygen style (\param my_var).
f) If a doxygen comment immediately precedes or follows the function definition or declaration, then the doxygen function declaration is redundant. Typically it just ends up causing problems because its argument list (types and names) must be kept in sync with those of both the definition (and declaration if separate). If it isn't kept in sync, doxygen's simple parser will complain that it can't find a matching function for the comment.
Change History (3)
comment:1 Changed 7 years ago by
comment:2 Changed 7 years ago by
Milestone: | needs_triage → 30.0 |
---|---|
Owner: | set to Stuart Auchterlonie |
Status: | new → assigned |
I'll need to upgrade the infrastructure to doxygen 1.8.13 before the config change part of this can be merged
comment:3 Changed 6 years ago by
Resolution: | → fixed |
---|---|
Status: | assigned → closed |
Patches are at https://github.com/MythTV/mythtv/pull/144