free impl_ in AbstractMetaObjectBase destructor

[yechun1]

The constructor allocates field impl_ of class_loader::impl::AbstractMetaObjectBase but the destructor and whatever functions it calls do not free it.

Signed-off-by: Chris Ye chris.ye@intel.com

[yechun1]

Just checked the CI fails all related with build system itself.

The command "python3 -u ./src/ament/ament_tools/scripts/ament.py build --build-tests --ament-cmake-args -DPYTHON_VERSION=3 -- --end-with class_loader" exited with 1.
8430.19s$ python3 -u ./src/ament/ament_tools/scripts/ament.py test --skip-build --skip-install --only class_loader
844Traceback (most recent call last):
845 File "./src/ament/ament_tools/scripts/ament.py", line 139, in
846 from ament_tools.package_types.ament import entry_point_data as ament_entry_point_data # noqa
847 File "/home/travis/build/ros/class_loader/build/src/ament/ament_tools/ament_tools/package_types/ament.py", line 15, in
848 from catkin_pkg.package import package_exists_at
849ImportError: No module named 'catkin_pkg'

[mikaelarguedas]

CI on ROS2 packages using it to make sure it doesn't have side effects.

• Linux
• Linux-aarch64
• macOS
• Windows

[mikaelarguedas]

lgtm, thanks @yechun1 for the contribution