-
-
Notifications
You must be signed in to change notification settings - Fork 115
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 errors in SurfaceMesh and DigitalConvexity #1534
Conversation
9ea3995
to
13bb3ca
Compare
```cpp [195/454] Building CXX object tests/geometry/volumes/CMakeFiles/testDigitalConvexity.dir/testDigitalConvexity.cpp.o In file included from ../tests/geometry/volumes/testDigitalConvexity.cpp:38: ../src/DGtal/geometry/volumes/DigitalConvexity.h:258:14: warning: unrecognized parameter passing direction, valid directions are '[in]', '[out]' and '[in,out]' [-Wdocumentation] * @param[inout] out the output stream where information is outputed. ^~~~~~~ ../src/DGtal/geometry/volumes/DigitalConvexity.h:270:14: warning: unrecognized parameter passing direction, valid directions are '[in]', '[out]' and '[in,out]' [-Wdocumentation] * @param[inout] out the output stream where information is outputed. ^~~~~~~ 2 warnings generated. [245/454] Building CXX object tests/geometry/surfaces/CMakeFiles/testShroudsRegularization.dir/testShroudsRegularization.cpp.o In file included from ../tests/geometry/surfaces/testShroudsRegularization.cpp:35: In file included from ../src/DGtal/helpers/Shortcuts.h:62: In file included from ../src/DGtal/shapes/MeshHelpers.h:51: ../src/DGtal/shapes/SurfaceMesh.h:164:16: warning: parameter 'itPos,itPosEnd' not found in the function declaration [-Wdocumentation] /// @param itPos,itPosEnd a range of iterators pointing on the positions of all the ^~~~~~~~~~~~~~ ../src/DGtal/shapes/SurfaceMesh.h:167:16: warning: parameter 'itVertices,itVerticesEnd' not found in the function declaration [-Wdocumentation] /// @param itVertices,itVerticesEnd a range of iterators pointing ^~~~~~~~~~~~~~~~~~~~~~~~ ../src/DGtal/shapes/SurfaceMesh.h:187:16: warning: parameter 'itPos,itPosEnd' not found in the function declaration [-Wdocumentation] /// @param itPos,itPosEnd a range of iterators pointing on the positions of all the ^~~~~~~~~~~~~~ ../src/DGtal/shapes/SurfaceMesh.h:190:16: warning: parameter 'itVertices,itVerticesEnd' not found in the function declaration [-Wdocumentation] /// @param itVertices,itVerticesEnd a range of iterators pointing ^~~~~~~~~~~~~~~~~~~~~~~~ In file included from ../tests/geometry/surfaces/testShroudsRegularization.cpp:36: ../src/DGtal/geometry/surfaces/ShroudsRegularization.h:63:11: warning: empty paragraph passed to '@see' command [-Wdocumentation] /// @see \ref moduleShrouds ~~~~^ 5 warnings generated. [247/454] Building CXX object tests/geometry/surfaces/CMakeFiles/testIntegralInvariantShortcuts.dir/testIntegralInvariantShortcuts.cpp.o In file included from ../tests/geometry/surfaces/testIntegralInvariantShortcuts.cpp:38: In file included from ../src/DGtal/helpers/Shortcuts.h:62: In file included from ../src/DGtal/shapes/MeshHelpers.h:51: ../src/DGtal/shapes/SurfaceMesh.h:164:16: warning: parameter 'itPos,itPosEnd' not found in the function declaration [-Wdocumentation] /// @param itPos,itPosEnd a range of iterators pointing on the positions of all the ^~~~~~~~~~~~~~ ../src/DGtal/shapes/SurfaceMesh.h:167:16: warning: parameter 'itVertices,itVerticesEnd' not found in the function declaration [-Wdocumentation] /// @param itVertices,itVerticesEnd a range of iterators pointing ^~~~~~~~~~~~~~~~~~~~~~~~ ../src/DGtal/shapes/SurfaceMesh.h:187:16: warning: parameter 'itPos,itPosEnd' not found in the function declaration [-Wdocumentation] /// @param itPos,itPosEnd a range of iterators pointing on the positions of all the ^~~~~~~~~~~~~~ ../src/DGtal/shapes/SurfaceMesh.h:190:16: warning: parameter 'itVertices,itVerticesEnd' not found in the function declaration [-Wdocumentation] /// @param itVertices,itVerticesEnd a range of iterators pointing ^~~~~~~~~~~~~~~~~~~~~~~~ In file included from ../tests/geometry/surfaces/testIntegralInvariantShortcuts.cpp:39: In file included from ../src/DGtal/helpers/ShortcutsGeometry.h:54: ../src/DGtal/dec/ATSolver2D.h:201:13: warning: empty paragraph passed to '@see' command [-Wdocumentation] /// @see \ref DiscreteExteriorCalculusFactory for creating calculus objects. ~~~~^ In file included from ../tests/geometry/surfaces/testIntegralInvariantShortcuts.cpp:39: ../src/DGtal/helpers/ShortcutsGeometry.h:1378:15: warning: empty paragraph passed to '@see' command [-Wdocumentation] /// @see \ref moduleAT ~~~~^ ../src/DGtal/helpers/ShortcutsGeometry.h:1433:15: warning: empty paragraph passed to '@see' command [-Wdocumentation] /// @see \ref moduleAT ~~~~^ ../src/DGtal/helpers/ShortcutsGeometry.h:1501:15: warning: empty paragraph passed to '@see' command [-Wdocumentation] /// @see \ref moduleAT ~~~~^ ../src/DGtal/helpers/ShortcutsGeometry.h:1556:15: warning: empty paragraph passed to '@see' command [-Wdocumentation] /// @see \ref moduleAT ~~~~^ 9 warnings generated. [253/454] Building CXX object tests/geometry/surfaces/CMakeFiles/testDigitalSurfaceRegularization.dir/testDigitalSurfaceRegularization.cpp.o In file included from ../tests/geometry/surfaces/testDigitalSurfaceRegularization.cpp:35: In file included from ../src/DGtal/helpers/Shortcuts.h:62: In file included from ../src/DGtal/shapes/MeshHelpers.h:51: ../src/DGtal/shapes/SurfaceMesh.h:164:16: warning: parameter 'itPos,itPosEnd' not found in the function declaration [-Wdocumentation] /// @param itPos,itPosEnd a range of iterators pointing on the positions of all the ^~~~~~~~~~~~~~ ../src/DGtal/shapes/SurfaceMesh.h:167:16: warning: parameter 'itVertices,itVerticesEnd' not found in the function declaration [-Wdocumentation] /// @param itVertices,itVerticesEnd a range of iterators pointing ^~~~~~~~~~~~~~~~~~~~~~~~ ../src/DGtal/shapes/SurfaceMesh.h:187:16: warning: parameter 'itPos,itPosEnd' not found in the function declaration [-Wdocumentation] /// @param itPos,itPosEnd a range of iterators pointing on the positions of all the ^~~~~~~~~~~~~~ ../src/DGtal/shapes/SurfaceMesh.h:190:16: warning: parameter 'itVertices,itVerticesEnd' not found in the function declaration [-Wdocumentation] /// @param itVertices,itVerticesEnd a range of iterators pointing ^~~~~~~~~~~~~~~~~~~~~~~~ In file included from ../tests/geometry/surfaces/testDigitalSurfaceRegularization.cpp:36: In file included from ../src/DGtal/helpers/ShortcutsGeometry.h:54: ../src/DGtal/dec/ATSolver2D.h:201:13: warning: empty paragraph passed to '@see' command [-Wdocumentation] /// @see \ref DiscreteExteriorCalculusFactory for creating calculus objects. ~~~~^ In file included from ../tests/geometry/surfaces/testDigitalSurfaceRegularization.cpp:36: ../src/DGtal/helpers/ShortcutsGeometry.h:1378:15: warning: empty paragraph passed to '@see' command [-Wdocumentation] /// @see \ref moduleAT ~~~~^ ../src/DGtal/helpers/ShortcutsGeometry.h:1433:15: warning: empty paragraph passed to '@see' command [-Wdocumentation] /// @see \ref moduleAT ~~~~^ ../src/DGtal/helpers/ShortcutsGeometry.h:1501:15: warning: empty paragraph passed to '@see' command [-Wdocumentation] /// @see \ref moduleAT ~~~~^ ../src/DGtal/helpers/ShortcutsGeometry.h:1556:15: warning: empty paragraph passed to '@see' command [-Wdocumentation] /// @see \ref moduleAT ~~~~^ 9 warnings generated. ```
13bb3ca
to
1b1febd
Compare
@@ -161,12 +161,13 @@ namespace DGtal | |||
/// @tparam RealPointIterator any forward iterator on RealPoint. | |||
/// @tparam VerticesIterator any forward iterator on the range of vertices defining a face. | |||
/// | |||
/// @param itPos,itPosEnd a range of iterators pointing on the positions of all the | |||
/// vertices of the mesh. | |||
/// @param itPos start of range of iterators pointing on the positions of vertices of the mesh |
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.
Weird. When you look at https://www.doxygen.nl/manual/commands.html#cmdparam
it says that you can comment multiple parameters using comma separated items.
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.
Otherwise your modif is fine.
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.
Indeed, that's confusing, maybe CI is using an old doxygen version, or a globlal doxygen option is missing? No idea.
The travis CI seems again blocked due to timeout, but it seems unrelated to doc warnings. |
Checklist
cmake
mode (otherwise, Travis C.I. will fail).