search

code-iai / ROSIntegration

Unreal Engine Plugin to enable ROS Support
MIT License
412 stars 133 forks source link

Fixed memory leak when publishing messages #162

Closed tsender closed 2 years ago

tsender commented 2 years ago

See https://github.com/code-iai/ROSIntegration/issues/161 for discussion.

Changes:

  1. Fixed bson_t creation in all ConvertOutgoingMessage(TSharedPtr<FROSBaseMsg> BaseMsg, bson_t** message) functions
  2. Fixed bson_t creation in ROSBridge::QueueMessage(const std::string& topic_name, int queue_size, ROSBridgePublishMsg& msg)
  3. Added std_msgs/Bool message and converters
  4. Filled in sensor_msgs/RegionOfInterest converters
  5. Updated README section "Implementing New Message Types" with info on how to avoid a memory leak in the converters
Sanic commented 2 years ago

Thank you for the comprehensive PR!