Skip to content

Switch from TinyXML to TinyXML2#52

Merged
mikaelarguedas merged 1 commit intoros:indigo-develfrom
rojkov:tinyxml2
Jul 25, 2017
Merged

Switch from TinyXML to TinyXML2#52
mikaelarguedas merged 1 commit intoros:indigo-develfrom
rojkov:tinyxml2

Conversation

@rojkov
Copy link
Copy Markdown
Contributor

@rojkov rojkov commented Jun 14, 2017

The library TinyXML is considered to be unmaintained and
since all future development is focused on TinyXML2 this
patch updates pluginlib to use TinyXML2.

@mikaelarguedas
Copy link
Copy Markdown
Member

Thanks @rojkov for the contribution.

This needs ros-visualization/qt_gui_core#96 to be merged. Looking if any other package is assuming that pluginlib uses tinyxml.

The library TinyXML is considered to be unmaintained and
since all future development is focused on TinyXML2 this
patch updates pluginlib to use TinyXML2.

Signed-off-by: Dmitry Rozhkov <dmitry.rozhkov@linux.intel.com>
@mikaelarguedas
Copy link
Copy Markdown
Member

ros-visualization/qt_gui_core#96 has been merges, we now need ros-visualization/qt_gui_core#96 and ros-visualization/qt_gui_core#97 to be integrated to the groovy-devel branch before merging and releasing this

@mikaelarguedas mikaelarguedas merged commit 04af213 into ros:indigo-devel Jul 25, 2017
mikaelarguedas added a commit that referenced this pull request Jul 27, 2017
mikaelarguedas added a commit that referenced this pull request Jul 27, 2017
mikaelarguedas pushed a commit that referenced this pull request Jul 27, 2017
The library TinyXML is considered to be unmaintained and
since all future development is focused on TinyXML2 this
patch updates pluginlib to use TinyXML2.

Signed-off-by: Dmitry Rozhkov <dmitry.rozhkov@linux.intel.com>
@mikaelarguedas mikaelarguedas mentioned this pull request Jul 27, 2017
mikaelarguedas added a commit that referenced this pull request Jul 27, 2017
* Switch from TinyXML to TinyXML2 (#52)

The library TinyXML is considered to be unmaintained and
since all future development is focused on TinyXML2 this
patch updates pluginlib to use TinyXML2.

Signed-off-by: Dmitry Rozhkov <dmitry.rozhkov@linux.intel.com>

* add invalid xml exception

* throw InvalidXMLException when plugin file is invalid
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants