fixing lifecycle subscription signatures

Signed-off-by: William Woodall <[email protected]>
ros2 · May 7, 2019 · d620145 · d620145
1 parent 136d5a5
commit d620145
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/rclcpp_lifecycle/include/rclcpp_lifecycle/lifecycle_node.hpp b/rclcpp_lifecycle/include/rclcpp_lifecycle/lifecycle_node.hpp
@@ -227,7 +227,7 @@ class LifecycleNode : public node_interfaces::LifecycleNodeInterface,
     typename Alloc = std::allocator<void>,
     typename SubscriptionT = rclcpp::Subscription<MessageT, Alloc>>
   [[deprecated(
-    "use create_subscription(const std::string &, CallbackT, const rclcpp::QoS &, ...) instead"
+    "use create_subscription(const std::string &, const rclcpp::QoS &, CallbackT, ...) instead"
   )]]
   std::shared_ptr<SubscriptionT>
   create_subscription(
@@ -262,14 +262,14 @@ class LifecycleNode : public node_interfaces::LifecycleNodeInterface,
     typename Alloc = std::allocator<void>,
     typename SubscriptionT = rclcpp::Subscription<MessageT, Alloc>>
   [[deprecated(
-    "use create_subscription(const std::string &, CallbackT, const rclcpp::QoS &, ...) instead"
+    "use create_subscription(const std::string &, const rclcpp::QoS &, CallbackT, ...) instead"
   )]]
   std::shared_ptr<SubscriptionT>
   create_subscription(
     const std::string & topic_name,
     size_t qos_history_depth,
     CallbackT && callback,
-    rclcpp::callback_group::CallbackGroup::SharedPtr group = nullptr,
+    rclcpp::callback_group::CallbackGroup::SharedPtr group,
     bool ignore_local_publications = false,
     typename rclcpp::message_memory_strategy::MessageMemoryStrategy<
       typename rclcpp::subscription_traits::has_message_type<CallbackT>::type, Alloc>::SharedPtr

diff --git a/rclcpp_lifecycle/test/test_lifecycle_node.cpp b/rclcpp_lifecycle/test/test_lifecycle_node.cpp
@@ -218,7 +218,7 @@ TEST_F(TestDefaultStateMachine, lifecycle_subscriber) {
 
   auto cb = [](const std::shared_ptr<lifecycle_msgs::msg::State> msg) {(void) msg;};
   auto lifecycle_sub =
-    test_node->create_subscription<lifecycle_msgs::msg::State>("~/empty", cb, 10);
+    test_node->create_subscription<lifecycle_msgs::msg::State>("~/empty", 10, cb);
 
   SUCCEED();
 }