Fixes from review.

Signed-off-by: Chris Lalancette <[email protected]>

Author: clalancette

rclcpp/include/rclcpp/strategies/message_pool_memory_strategy.hpp

@@ -80,7 +80,7 @@ class MessagePoolMemoryStrategy
         }
       }
       if (free_list_.size() == 0) {
-        throw std::runtime_error("No more free slots in the pool!");
+        throw std::runtime_error("No more free slots in the pool");
       }
     }
 

rclcpp/test/rclcpp/strategies/test_message_pool_memory_strategy.cpp

@@ -56,7 +56,7 @@ TEST_F(TestMessagePoolMemoryStrategy, borrow_too_many) {
   // Size is 1, borrowing second time should fail
   RCLCPP_EXPECT_THROW_EQ(
     message_memory_strategy_->borrow_message(),
-    std::runtime_error("No more free slots in the pool!"));
+    std::runtime_error("No more free slots in the pool"));
   EXPECT_NO_THROW(message_memory_strategy_->return_message(message));
 }
 
@@ -71,7 +71,7 @@ TEST_F(TestMessagePoolMemoryStrategy, borrow_hold_reference) {
     // But we are still holding the reference, so we expect that there is still no room in the pool.
     RCLCPP_EXPECT_THROW_EQ(
       message_memory_strategy_->borrow_message(),
-      std::runtime_error("No more free slots in the pool!"));
+      std::runtime_error("No more free slots in the pool"));
   }
 
   // Now that we've dropped the reference (left the scope), we expect to be able to borrow again.