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
Style fixes from ros2 #11
Conversation
Signed-off-by: Chris Lalancette <clalancette@openrobotics.org>
Signed-off-by: Chris Lalancette <clalancette@openrobotics.org>
@@ -30,6 +30,10 @@ target_link_libraries(${PROJECT_NAME} | |||
${TinyXML_LIBRARIES} ${orocos_kdl_LIBRARIES} ${catkin_LIBRARIES} | |||
) | |||
|
|||
if(WIN32) | |||
target_compile_definitions(${PROJECT_NAME} PRIVATE "KDL_PARSER_BUILDING_DLL") |
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.
GenerateExportHeader?
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 basic idea as ros/urdf#11, where William's feedback is interesting. I can open an issue here as well.
kdl_parser/src/check_kdl_parser.cpp
Outdated
|
||
void printLink(const SegmentMap::const_iterator& link, const std::string& prefix) | ||
void printLink(const KDL::SegmentMap::const_iterator & link, const std::string & prefix) |
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.
If qualifying with KDL::
then should using namespace KDL;
online 45 be deleted?
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.
Good call, done now.
kdl_parser/test/test_inertia_rpy.cpp
Outdated
* POSSIBILITY OF SUCH DAMAGE. | ||
*********************************************************************/ | ||
|
||
/* Author: Wim Meeussen */ |
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.
The author of this file looks like it might be @jacquelinekay in ed6237c
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.
Yeah, good call. That was a cut-n-paste from elsewhere in the library, but fixed now.
Signed-off-by: Chris Lalancette <clalancette@openrobotics.org>
Signed-off-by: Chris Lalancette <clalancette@openrobotics.org>
* Style fixes (these can go to upstream kdl_parser) Signed-off-by: Chris Lalancette <clalancette@openrobotics.org> * Windows compatibility. Signed-off-by: Chris Lalancette <clalancette@openrobotics.org>
Pretty much what it says on the tin. Import some style fixes from https://github.com/ros2/kdl_parser to reduce the difference between them. Should be no functional change.