From 0723a0a6fc2f46edcbeebf2933c7929822177d56 Mon Sep 17 00:00:00 2001 From: Alberto Soragna Date: Tue, 28 May 2019 22:52:32 +0100 Subject: [PATCH] removed not used parameter client (#740) * removed not used parameter client Signed-off-by: alberto * moved parameter include directives to time source cpp file Signed-off-by: alberto --- rclcpp/include/rclcpp/time_source.hpp | 5 ----- rclcpp/src/rclcpp/time_source.cpp | 4 +++- 2 files changed, 3 insertions(+), 6 deletions(-) diff --git a/rclcpp/include/rclcpp/time_source.hpp b/rclcpp/include/rclcpp/time_source.hpp index 09a8340383..4dc46ce3e6 100644 --- a/rclcpp/include/rclcpp/time_source.hpp +++ b/rclcpp/include/rclcpp/time_source.hpp @@ -25,8 +25,6 @@ #include "rcl_interfaces/msg/parameter_event.hpp" #include "rclcpp/node.hpp" -#include "rclcpp/parameter_client.hpp" -#include "rclcpp/parameter_events_filter.hpp" namespace rclcpp @@ -100,9 +98,6 @@ class TimeSource // Destroy the subscription for the clock topic void destroy_clock_sub(); - // Parameter Client pointer - std::shared_ptr parameter_client_; - // Parameter Event subscription using ParamMessageT = rcl_interfaces::msg::ParameterEvent; using ParamSubscriptionT = rclcpp::Subscription; diff --git a/rclcpp/src/rclcpp/time_source.cpp b/rclcpp/src/rclcpp/time_source.cpp index 3639e104de..db8dd7ed15 100644 --- a/rclcpp/src/rclcpp/time_source.cpp +++ b/rclcpp/src/rclcpp/time_source.cpp @@ -25,6 +25,8 @@ #include "rclcpp/exceptions.hpp" #include "rclcpp/logging.hpp" #include "rclcpp/node.hpp" +#include "rclcpp/parameter_client.hpp" +#include "rclcpp/parameter_events_filter.hpp" #include "rclcpp/time.hpp" #include "rclcpp/time_source.hpp" @@ -112,7 +114,7 @@ void TimeSource::detachNode() { this->ros_time_active_ = false; clock_subscription_.reset(); - parameter_client_.reset(); + parameter_subscription_.reset(); node_base_.reset(); node_topics_.reset(); node_graph_.reset();