ros / urdfdom

URDF parser
http://ros.org/wiki/urdf
Other
96 stars 132 forks source link

Deprecate the APIs that we think are unused. #191

Closed clalancette closed 10 months ago

clalancette commented 10 months ago

In an earlier commit, we changed from tinyxml -> tinyxml2 in the public API because we thought that there were no users of these APIs. Codify that here by marking these APIs as deprecated; if a user comes along and says that they are actually using it, we can undeprecated it.

Note that in order to avoid deprecations from within the library, I had to add a bit of additional indirection here. If we remove the APIs in the future, we can also remove this indirection.

@scpeters @sloretz FYI.

clalancette commented 10 months ago

CI:

clalancette commented 10 months ago

New CI with a fix for Windows: