BUG: cv::Mat needs a deep copy with clone()

ros-drivers / video_stream_opencv

A package to open video streams and publish them in ROS using the opencv videocapture mechanism

227 stars 159 forks source link

BUG: cv::Mat needs a deep copy with clone() #22

Closed Axel13fr closed 6 years ago

Axel13fr commented 6 years ago

BUG: cv::Mat needs to be cloned otherwise all entries in the queue will point to the same frame (mat.data is a pointer and its content is only copied when using clone). This resulted in always returning the last received frame.

Axel13fr commented 6 years ago

@awesomebytes I forgot to clone matrices in my previous fix !

awesomebytes commented 6 years ago

Re-release requested on all distros. Thank you for fixing. That was bad!