From 0fd8cbe6fba6e8870d2437849c2c0de0cf5889bd Mon Sep 17 00:00:00 2001 From: Dirk Thomas Date: Mon, 20 Jul 2015 15:52:25 -0700 Subject: [PATCH] raise warning level --- rclcpp/include/rclcpp/client.hpp | 2 +- rclcpp/include/rclcpp/memory_strategy.hpp | 2 ++ rclcpp/include/rclcpp/parameter_service.hpp | 12 ++++++------ .../rclcpp/strategies/static_memory_strategy.hpp | 1 + rclcpp/include/rclcpp/utilities.hpp | 2 ++ 5 files changed, 12 insertions(+), 7 deletions(-) diff --git a/rclcpp/include/rclcpp/client.hpp b/rclcpp/include/rclcpp/client.hpp index 7bf3921..c41963f 100644 --- a/rclcpp/include/rclcpp/client.hpp +++ b/rclcpp/include/rclcpp/client.hpp @@ -138,7 +138,7 @@ public: SharedFuture async_send_request( typename ServiceT::Request::SharedPtr & request) { - return async_send_request(request, [](SharedFuture f) {}); + return async_send_request(request, [](SharedFuture) {}); } SharedFuture async_send_request( diff --git a/rclcpp/include/rclcpp/memory_strategy.hpp b/rclcpp/include/rclcpp/memory_strategy.hpp index 6d81bce..fa5e26d 100644 --- a/rclcpp/include/rclcpp/memory_strategy.hpp +++ b/rclcpp/include/rclcpp/memory_strategy.hpp @@ -39,11 +39,13 @@ public: RCLCPP_MAKE_SHARED_DEFINITIONS(MemoryStrategy); virtual void ** borrow_handles(HandleType type, size_t number_of_handles) { + (void)type; return static_cast(alloc(sizeof(void *) * number_of_handles)); } virtual void return_handles(HandleType type, void ** handles) { + (void)type; this->free(handles); } diff --git a/rclcpp/include/rclcpp/parameter_service.hpp b/rclcpp/include/rclcpp/parameter_service.hpp index 66b1859..9bae3ec 100644 --- a/rclcpp/include/rclcpp/parameter_service.hpp +++ b/rclcpp/include/rclcpp/parameter_service.hpp @@ -48,7 +48,7 @@ public: { get_parameters_service_ = node_->create_service( node_->get_name() + "__get_parameters", [&node]( - const std::shared_ptr request_header, + const std::shared_ptr, const std::shared_ptr request, std::shared_ptr response) { @@ -63,7 +63,7 @@ public: get_parameter_types_service_ = node_->create_service( node_->get_name() + "__get_parameter_types", [&node]( - const std::shared_ptr request_header, + const std::shared_ptr, const std::shared_ptr request, std::shared_ptr response) { @@ -77,7 +77,7 @@ public: set_parameters_service_ = node_->create_service( node_->get_name() + "__set_parameters", [&node]( - const std::shared_ptr request_header, + const std::shared_ptr, const std::shared_ptr request, std::shared_ptr response) { @@ -96,7 +96,7 @@ public: set_parameters_atomically_service_ = node_->create_service( node_->get_name() + "__set_parameters_atomically", [&node]( - const std::shared_ptr request_header, + const std::shared_ptr, const std::shared_ptr request, std::shared_ptr response) { @@ -114,7 +114,7 @@ public: describe_parameters_service_ = node_->create_service( node_->get_name() + "__describe_parameters", [&node]( - const std::shared_ptr request_header, + const std::shared_ptr, const std::shared_ptr request, std::shared_ptr response) { @@ -125,7 +125,7 @@ public: list_parameters_service_ = node_->create_service( node_->get_name() + "__list_parameters", [&node]( - const std::shared_ptr request_header, + const std::shared_ptr, const std::shared_ptr request, std::shared_ptr response) { diff --git a/rclcpp/include/rclcpp/strategies/static_memory_strategy.hpp b/rclcpp/include/rclcpp/strategies/static_memory_strategy.hpp index 627ab1c..93b6dc8 100644 --- a/rclcpp/include/rclcpp/strategies/static_memory_strategy.hpp +++ b/rclcpp/include/rclcpp/strategies/static_memory_strategy.hpp @@ -88,6 +88,7 @@ public: void return_handles(HandleType type, void ** handles) { + (void)handles; switch (type) { case HandleType::subscriber_handle: memset(subscriber_pool_, 0, max_subscribers_); diff --git a/rclcpp/include/rclcpp/utilities.hpp b/rclcpp/include/rclcpp/utilities.hpp index 5968683..eb6db5c 100644 --- a/rclcpp/include/rclcpp/utilities.hpp +++ b/rclcpp/include/rclcpp/utilities.hpp @@ -95,6 +95,8 @@ namespace utilities void init(int argc, char * argv[]) { + (void)argc; + (void)argv; // TODO(wjwwood): Handle rmw_ret_t's value. rmw_init(); #ifdef HAS_SIGACTION