-
Notifications
You must be signed in to change notification settings - Fork 106
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 rosdoc2 warnings in sensor_msgs. #162
Conversation
Make sure the build of documentation is clean. No functional changes in here, though there are some formatting changes to make things easier to document. Signed-off-by: Chris Lalancette <clalancette@openrobotics.org>
/** | ||
* @brief Function setting some fields in a PointCloud and adjusting the | ||
* internals of the PointCloud2 | ||
* @param n_fields the number of fields to add. The fields are given as | ||
* triplets: name of the field as char*, number of elements in the | ||
* field, the datatype of the elements in the field | ||
* | ||
* E.g, you create your PointCloud2 message with XYZ/RGB as follows: | ||
* <PRE> | ||
* setPointCloud2FieldsByString(cloud_msg, 4, "x", 1, sensor_msgs::msg::PointField::FLOAT32, | ||
* "y", 1, sensor_msgs::msg::PointField::FLOAT32, | ||
* "z", 1, sensor_msgs::msg::PointField::FLOAT32, | ||
* "rgb", 1, sensor_msgs::msg::PointField::FLOAT32); | ||
* </PRE> | ||
* WARNING: THIS DOES NOT TAKE INTO ACCOUNT ANY PADDING AS DONE UNTIL HYDRO | ||
* For simple usual cases, the overloaded setPointCloud2FieldsByString is what you want. | ||
*/ |
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.
Is this documentation block removed because it's an implementation file?
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.
Yes, exactly. Doxygen was complaining that n_fields
was documented twice, and that was because we had the same docblock on the declaration and definition. Removing one of them gets rid of that warning.
/** | ||
* @brief Function setting some fields in a PointCloud and adjusting the | ||
* internals of the PointCloud2 | ||
* @param n_fields the number of fields to add. The fields are given as | ||
* strings: "xyz" (3 floats), "rgb" (3 uchar stacked in a float), | ||
* "rgba" (4 uchar stacked in a float) | ||
* @return void | ||
* | ||
* WARNING: THIS FUNCTION DOES ADD ANY NECESSARY PADDING TRANSPARENTLY | ||
*/ |
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.
Is this documentation block removed because it's an implementation file?
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.
Same answer as above.
Since this is "just" a change to the comments (and a bit of formatting fixes), I'm going to merge it just based on the Rpr results. Thanks for the review! |
Make sure the build of documentation is clean. No functional
changes in here, though there are some formatting changes to make
things easier to document.
Signed-off-by: Chris Lalancette clalancette@openrobotics.org