ros-perception / image_common

Common code for working with images in ROS

Home Page:http://www.ros.org/wiki/image_common

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

SimpleSubscriberPlugin shutdown() implementation

afrixs opened this issue · comments

There is a TODO in SimpleSubscriberPlugin about waiting for ROS2 to support subscriber shutdown.

I believe ROS2 now supports subscriber shutdown - it's performed in subscriber's destructor.

I would propose to simply use impl_.reset(); like it's done e.g. in message_filters package. I've tested it and it works.

Could you please submit a PR that includes a test? We can help you through the process if you're unsure how to do one.

Reopening since calling publish on a disabled publisher caused process to crash (see #235)