View Issue Details
ID | Project | Category | View Status | Date Submitted | Last Update |
---|---|---|---|---|---|
0002023 | OpenFOAM | Patch | public | 2016-03-14 01:34 | 2016-03-14 11:04 |
Reporter | wyldckat | Assigned To | henry | ||
Priority | normal | Severity | text | Reproducibility | N/A |
Status | resolved | Resolution | fixed | ||
Product Version | dev | ||||
Summary | 0002023: Code documentation: Several file updates, including CSS and header file | ||||
Description | So I went through a long rabbit hole today, when I was only planning on providing a fix for "uniformInterpolatedDisplacement", which for 3.0.1 Doxygen rendered everything in HTML as if it was a straight line. Attached is the file "tweakingCodeDoc_v1.tar.gz", which provides the following changes: 1- Several Doxygen-compatible (tested with Doxygen 1.8.9 on Ubuntu 15.10) format repairs to the descriptions of several class header files. This included: - broken lists, due to inconsistent Doxygen parsing on what constitutes a list or not... - missing verbatim blocks or inconsistent indentation in those blocks; - a few changes from quotes to \c command; - added a few of the missing parameter descriptions, for which Doxygen warned about it being incomplete. 2- "doc/Doxygen/_Header": Added missing JavaScript files and jumpstart call for the search function to work properly again; and fixed typo in the first <meta> block. 3- "doc/Doxygen/css/cfdLayout.css": Added font format to <code> blocks and removed non-existent background image link. 4- Removed extra include for "PstreamReduceOps.H", pointed out by Doxygen with a cryptic message of "could not find file" (can't remember the exact statement). 5- Fixed proper including of files in "src/OpenFOAM/graph/curve/curveTools.H" (same cryptic message was given by Doxygen). 6- A few changes from \a to \c 7- Repaired usage of "\cond ignoreDocumentation" in a couple of files. 8- Fixed a few typos here and there, as well as removed/replaced a few repeated descriptions about "kappa" and "kappaName" that are already explained in another class. 9- Added a few missing SeeAlso and SourceFiles. 10- Repaired a few class paths in the header files. 11- RASBoundaryConditionsDoc.H - repaired overriding name of group "grpWallFunctions", by renaming it to "grpRASWallFunctions". | ||||
Additional Information | Side note: The class "alphaContactAngleFvPatchScalarField" is a curious example of what happens if we have several header files with the same name and namespace location: http://foam.sourceforge.net/docs/cpp/a00037.html#details The result is that the comments for all 5 different instances are all documented into the same page in HTML format. So, either these should be placed in different namespaces or some other strategy should be used with Doxygen grouping? | ||||
Tags | No tags attached. | ||||
|
|
|
Thanks Bruno Resolved by commit 7037737d03d1d8464d3ba897070c04d7d2bd42b1 Update online documentation http://openfoam.github.io/Documentation-dev/html/ |
Date Modified | Username | Field | Change |
---|---|---|---|
2016-03-14 01:34 | wyldckat | New Issue | |
2016-03-14 01:34 | wyldckat | Status | new => assigned |
2016-03-14 01:34 | wyldckat | Assigned To | => henry |
2016-03-14 01:34 | wyldckat | File Added: tweakingCodeDoc_v1.tar.gz | |
2016-03-14 11:03 | henry | Note Added: 0006034 | |
2016-03-14 11:03 | henry | Status | assigned => resolved |
2016-03-14 11:03 | henry | Resolution | open => fixed |