Skip to content

Commit 1a9b117

Browse files
mauropasseMauro Passerino
andauthored
Fix clang warning: bugprone-use-after-move (#2116)
Signed-off-by: Mauro Passerino <[email protected]> Co-authored-by: Mauro Passerino <[email protected]>
1 parent 11778f5 commit 1a9b117

File tree

1 file changed

+4
-0
lines changed

1 file changed

+4
-0
lines changed

rclcpp/include/rclcpp/experimental/intra_process_manager.hpp

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -454,6 +454,8 @@ class IntraProcessManager
454454
if (std::next(it) == subscription_ids.end()) {
455455
// If this is the last subscription, give up ownership
456456
subscription->provide_intra_process_data(std::move(message));
457+
// Last message delivered, break from for loop
458+
break;
457459
} else {
458460
// Copy the message since we have additional subscriptions to serve
459461
Deleter deleter = message.get_deleter();
@@ -493,6 +495,8 @@ class IntraProcessManager
493495
if (std::next(it) == subscription_ids.end()) {
494496
// If this is the last subscription, give up ownership
495497
ros_message_subscription->provide_intra_process_message(std::move(message));
498+
// Last message delivered, break from for loop
499+
break;
496500
} else {
497501
// Copy the message since we have additional subscriptions to serve
498502
Deleter deleter = message.get_deleter();

0 commit comments

Comments
 (0)