Skip to content
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

rviz is using deprecated tinyxml interface of URDF #1235

Closed
wjwwood opened this issue May 9, 2018 · 0 comments · Fixed by #1237
Closed

rviz is using deprecated tinyxml interface of URDF #1235

wjwwood opened this issue May 9, 2018 · 0 comments · Fixed by #1237
Assignees
Labels

Comments

@wjwwood
Copy link
Member

wjwwood commented May 9, 2018

It produces a warning like this in melodic:

[ 81%] Building CXX object src/rviz/default_plugin/CMakeFiles/rviz_default_plugin.dir/robot_model_display.cpp.o
/home/william/melodic/src/rviz/src/rviz/default_plugin/robot_model_display.cpp: In member function ‘virtual void rviz::RobotModelDisplay::load()’:
/home/william/melodic/src/rviz/src/rviz/default_plugin/robot_model_display.cpp:185:41: warning: ‘bool urdf::Model::initXml(TiXmlElement*)’ is deprecated: TinyXML API is deprecated, use the TinyXML2 version instead [-Wdeprecated-declarations]
   if( !descr.initXml( doc.RootElement() ))
                                         ^
In file included from /home/william/melodic/src/rviz/src/rviz/default_plugin/robot_model_display.cpp:34:0:
/home/william/melodic/install/include/urdf/model.h:61:99: note: declared here
   URDF_EXPORT URDF_DEPRECATED("TinyXML API is deprecated, use the TinyXML2 version instead") bool initXml(TiXmlElement * xml);
                                                                                                   ^~~~~~~
@wjwwood wjwwood added the bug label May 9, 2018
@wjwwood wjwwood self-assigned this May 9, 2018
@wjwwood wjwwood added the review label May 10, 2018
@wjwwood wjwwood added this to the melodic release milestone May 10, 2018
@ghost ghost removed the review label May 10, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant