Fix include order and relative paths for cpplint

Relates to ament/ament_lint#324 Signed-off-by: Jacob Perron <[email protected]>
ros2 · Jan 7, 2022 · 733f955 · 733f955
1 parent 8093648
commit 733f955
Show file tree

Hide file tree

Showing 17 changed files with 93 additions and 92 deletions.
diff --git a/rclcpp/src/rclcpp/detail/resolve_parameter_overrides.cpp b/rclcpp/src/rclcpp/detail/resolve_parameter_overrides.cpp
@@ -12,7 +12,7 @@
 // See the License for the specific language governing permissions and
 // limitations under the License.
 
-#include "./resolve_parameter_overrides.hpp"
+#include "resolve_parameter_overrides.hpp"
 
 #include <string>
 #include <map>

diff --git a/rclcpp/src/rclcpp/executors/multi_threaded_executor.cpp b/rclcpp/src/rclcpp/executors/multi_threaded_executor.cpp
@@ -72,8 +72,9 @@ MultiThreadedExecutor::get_number_of_threads()
 }
 
 void
-MultiThreadedExecutor::run(size_t)
+MultiThreadedExecutor::run(size_t this_thread_number)
 {
+  (void)this_thread_number;
   while (rclcpp::ok(this->context_) && spinning.load()) {
     rclcpp::AnyExecutable any_exec;
     {

diff --git a/rclcpp/src/rclcpp/signal_handler.cpp b/rclcpp/src/rclcpp/signal_handler.cpp
@@ -12,7 +12,7 @@
 // See the License for the specific language governing permissions and
 // limitations under the License.
 
-#include "./signal_handler.hpp"
+#include "signal_handler.hpp"
 
 #include <atomic>
 #include <csignal>

diff --git a/rclcpp/test/rclcpp/test_intra_process_manager.cpp b/rclcpp/test/rclcpp/test_intra_process_manager.cpp
@@ -301,7 +301,7 @@ class SubscriptionIntraProcess : public SubscriptionIntraProcessBuffer<
 #define SubscriptionIntraProcessBase mock::SubscriptionIntraProcessBase
 #define SubscriptionIntraProcessBuffer mock::SubscriptionIntraProcessBuffer
 #define SubscriptionIntraProcess mock::SubscriptionIntraProcess
-#include "../src/rclcpp/intra_process_manager.cpp"
+#include "../src/rclcpp/intra_process_manager.cpp"  // NOLINT
 #undef Publisher
 #undef PublisherBase
 #undef IntraProcessBuffer

diff --git a/rclcpp_action/include/rclcpp_action/client.hpp b/rclcpp_action/include/rclcpp_action/client.hpp
@@ -15,6 +15,16 @@
 #ifndef RCLCPP_ACTION__CLIENT_HPP_
 #define RCLCPP_ACTION__CLIENT_HPP_
 
+#include <algorithm>
+#include <chrono>
+#include <functional>
+#include <future>
+#include <map>
+#include <memory>
+#include <mutex>
+#include <string>
+#include <utility>
+
 #include <rclcpp/exceptions.hpp>
 #include <rclcpp/macros.hpp>
 #include <rclcpp/node_interfaces/node_base_interface.hpp>
@@ -24,19 +34,9 @@
 #include <rclcpp/time.hpp>
 #include <rclcpp/waitable.hpp>
 
-#include <rosidl_runtime_c/action_type_support_struct.h>
+#include "rosidl_runtime_c/action_type_support_struct.h"
 #include <rosidl_typesupport_cpp/action_type_support.hpp>
 
-#include <algorithm>
-#include <chrono>
-#include <functional>
-#include <future>
-#include <map>
-#include <memory>
-#include <mutex>
-#include <string>
-#include <utility>
-
 #include "rclcpp_action/client_goal_handle.hpp"
 #include "rclcpp_action/exceptions.hpp"
 #include "rclcpp_action/types.hpp"

diff --git a/rclcpp_action/include/rclcpp_action/client_goal_handle.hpp b/rclcpp_action/include/rclcpp_action/client_goal_handle.hpp
@@ -15,17 +15,17 @@
 #ifndef RCLCPP_ACTION__CLIENT_GOAL_HANDLE_HPP_
 #define RCLCPP_ACTION__CLIENT_GOAL_HANDLE_HPP_
 
-#include <rcl_action/action_client.h>
-
-#include <action_msgs/msg/goal_status.hpp>
-#include <rclcpp/macros.hpp>
-#include <rclcpp/time.hpp>
-
 #include <functional>
 #include <future>
 #include <memory>
 #include <mutex>
 
+#include "rcl_action/action_client.h"
+
+#include <action_msgs/msg/goal_status.hpp>
+#include <rclcpp/macros.hpp>
+#include <rclcpp/time.hpp>
+
 #include "rclcpp_action/exceptions.hpp"
 #include "rclcpp_action/types.hpp"
 #include "rclcpp_action/visibility_control.hpp"

diff --git a/rclcpp_action/include/rclcpp_action/create_client.hpp b/rclcpp_action/include/rclcpp_action/create_client.hpp
@@ -15,11 +15,11 @@
 #ifndef RCLCPP_ACTION__CREATE_CLIENT_HPP_
 #define RCLCPP_ACTION__CREATE_CLIENT_HPP_
 
-#include <rclcpp/node.hpp>
-
 #include <memory>
 #include <string>
 
+#include <rclcpp/node.hpp>
+
 #include "rclcpp_action/client.hpp"
 #include "rclcpp_action/visibility_control.hpp"
 

diff --git a/rclcpp_action/include/rclcpp_action/create_server.hpp b/rclcpp_action/include/rclcpp_action/create_server.hpp
@@ -15,17 +15,17 @@
 #ifndef RCLCPP_ACTION__CREATE_SERVER_HPP_
 #define RCLCPP_ACTION__CREATE_SERVER_HPP_
 
-#include <rcl_action/action_server.h>
+#include <memory>
+#include <string>
+
+#include "rcl_action/action_server.h"
 
 #include <rclcpp/node.hpp>
 #include <rclcpp/node_interfaces/node_base_interface.hpp>
 #include <rclcpp/node_interfaces/node_clock_interface.hpp>
 #include <rclcpp/node_interfaces/node_logging_interface.hpp>
 #include <rclcpp/node_interfaces/node_waitables_interface.hpp>
 
-#include <memory>
-#include <string>
-
 #include "rclcpp_action/server.hpp"
 #include "rclcpp_action/visibility_control.hpp"
 

diff --git a/rclcpp_action/include/rclcpp_action/server.hpp b/rclcpp_action/include/rclcpp_action/server.hpp
@@ -15,21 +15,21 @@
 #ifndef RCLCPP_ACTION__SERVER_HPP_
 #define RCLCPP_ACTION__SERVER_HPP_
 
-#include <rcl_action/action_server.h>
-#include <rosidl_runtime_c/action_type_support_struct.h>
-#include <rosidl_typesupport_cpp/action_type_support.hpp>
-#include <rclcpp/node_interfaces/node_base_interface.hpp>
-#include <rclcpp/node_interfaces/node_clock_interface.hpp>
-#include <rclcpp/node_interfaces/node_logging_interface.hpp>
-#include <rclcpp/waitable.hpp>
-
 #include <functional>
 #include <memory>
 #include <mutex>
 #include <string>
 #include <unordered_map>
 #include <utility>
 
+#include "rcl_action/action_server.h"
+#include "rosidl_runtime_c/action_type_support_struct.h"
+#include <rosidl_typesupport_cpp/action_type_support.hpp>
+#include <rclcpp/node_interfaces/node_base_interface.hpp>
+#include <rclcpp/node_interfaces/node_clock_interface.hpp>
+#include <rclcpp/node_interfaces/node_logging_interface.hpp>
+#include <rclcpp/waitable.hpp>
+
 #include "rclcpp_action/visibility_control.hpp"
 #include "rclcpp_action/server_goal_handle.hpp"
 #include "rclcpp_action/types.hpp"

diff --git a/rclcpp_action/include/rclcpp_action/server_goal_handle.hpp b/rclcpp_action/include/rclcpp_action/server_goal_handle.hpp
@@ -15,15 +15,15 @@
 #ifndef RCLCPP_ACTION__SERVER_GOAL_HANDLE_HPP_
 #define RCLCPP_ACTION__SERVER_GOAL_HANDLE_HPP_
 
-#include <rcl_action/types.h>
-#include <rcl_action/goal_handle.h>
-
-#include <action_msgs/msg/goal_status.hpp>
-
 #include <functional>
 #include <memory>
 #include <mutex>
 
+#include "rcl_action/types.h"
+#include "rcl_action/goal_handle.h"
+
+#include <action_msgs/msg/goal_status.hpp>
+
 #include "rclcpp_action/visibility_control.hpp"
 #include "rclcpp_action/types.hpp"
 

diff --git a/rclcpp_action/include/rclcpp_action/types.hpp b/rclcpp_action/include/rclcpp_action/types.hpp
@@ -15,15 +15,15 @@
 #ifndef RCLCPP_ACTION__TYPES_HPP_
 #define RCLCPP_ACTION__TYPES_HPP_
 
-#include <rcl_action/types.h>
-
-#include <action_msgs/msg/goal_status.hpp>
-#include <action_msgs/msg/goal_info.hpp>
-
 #include <climits>
 #include <functional>
 #include <string>
 
+#include "rcl_action/types.h"
+
+#include <action_msgs/msg/goal_status.hpp>
+#include <action_msgs/msg/goal_info.hpp>
+
 #include "rclcpp_action/visibility_control.hpp"
 
 namespace rclcpp_action

diff --git a/rclcpp_action/src/client.cpp b/rclcpp_action/src/client.cpp
@@ -12,11 +12,6 @@
 // See the License for the specific language governing permissions and
 // limitations under the License.
 
-#include <rcl_action/action_client.h>
-#include <rcl_action/wait.h>
-#include <rclcpp/node_interfaces/node_base_interface.hpp>
-#include <rclcpp/node_interfaces/node_logging_interface.hpp>
-
 #include <algorithm>
 #include <map>
 #include <memory>
@@ -25,6 +20,11 @@
 #include <tuple>
 #include <utility>
 
+#include "rcl_action/action_client.h"
+#include "rcl_action/wait.h"
+#include <rclcpp/node_interfaces/node_base_interface.hpp>
+#include <rclcpp/node_interfaces/node_logging_interface.hpp>
+
 #include "rclcpp_action/client.hpp"
 #include "rclcpp_action/exceptions.hpp"
 

diff --git a/rclcpp_action/src/server.cpp b/rclcpp_action/src/server.cpp
@@ -12,16 +12,6 @@
 // See the License for the specific language governing permissions and
 // limitations under the License.
 
-#include <rcl_action/action_server.h>
-#include <rcl_action/wait.h>
-
-#include <rcpputils/scope_exit.hpp>
-
-#include <action_msgs/msg/goal_status_array.hpp>
-#include <action_msgs/srv/cancel_goal.hpp>
-#include <rclcpp/exceptions.hpp>
-#include <rclcpp_action/server.hpp>
-
 #include <memory>
 #include <mutex>
 #include <string>
@@ -30,6 +20,16 @@
 #include <utility>
 #include <vector>
 
+#include "rcl_action/action_server.h"
+#include "rcl_action/wait.h"
+
+#include <rcpputils/scope_exit.hpp>
+
+#include <action_msgs/msg/goal_status_array.hpp>
+#include <action_msgs/srv/cancel_goal.hpp>
+#include <rclcpp/exceptions.hpp>
+#include <rclcpp_action/server.hpp>
+
 using rclcpp_action::ServerBase;
 using rclcpp_action::GoalUUID;
 

diff --git a/rclcpp_action/src/server_goal_handle.cpp b/rclcpp_action/src/server_goal_handle.cpp
@@ -12,14 +12,14 @@
 // See the License for the specific language governing permissions and
 // limitations under the License.
 
-#include <rcl_action/action_server.h>
-#include <rcl_action/goal_handle.h>
+#include <memory>
+
+#include "rcl_action/action_server.h"
+#include "rcl_action/goal_handle.h"
 
 #include <rclcpp_action/server_goal_handle.hpp>
 #include <rclcpp/exceptions.hpp>
 
-#include <memory>
-
 namespace rclcpp_action
 {
 ServerGoalHandleBase::~ServerGoalHandleBase()

diff --git a/rclcpp_action/test/test_client.cpp b/rclcpp_action/test/test_client.cpp
@@ -12,15 +12,23 @@
 // See the License for the specific language governing permissions and
 // limitations under the License.
 
-#include <gtest/gtest.h>
+#include <future>
+#include <map>
+#include <memory>
+#include <string>
+#include <utility>
+#include <thread>
+#include <chrono>
+
+#include "gtest/gtest.h"
 
-#include <rcl/allocator.h>
-#include <rcl/time.h>
-#include <rcl/types.h>
+#include "rcl/allocator.h"
+#include "rcl/time.h"
+#include "rcl/types.h"
 
-#include <rcl_action/names.h>
-#include <rcl_action/default_qos.h>
-#include <rcl_action/wait.h>
+#include "rcl_action/names.h"
+#include "rcl_action/default_qos.h"
+#include "rcl_action/wait.h"
 
 #include <rclcpp/clock.hpp>
 #include <rclcpp/exceptions.hpp>
@@ -33,21 +41,13 @@
 
 #include <test_msgs/action/fibonacci.hpp>
 
-#include <future>
-#include <map>
-#include <memory>
-#include <string>
-#include <utility>
-#include <thread>
-#include <chrono>
-
 #include "rclcpp_action/exceptions.hpp"
 #include "rclcpp_action/create_client.hpp"
 #include "rclcpp_action/client.hpp"
 #include "rclcpp_action/qos.hpp"
 #include "rclcpp_action/types.hpp"
 
-#include "./mocking_utils/patch.hpp"
+#include "mocking_utils/patch.hpp"
 
 using namespace std::chrono_literals;
 

diff --git a/rclcpp_action/test/test_server.cpp b/rclcpp_action/test/test_server.cpp
@@ -12,22 +12,22 @@
 // See the License for the specific language governing permissions and
 // limitations under the License.
 
+#include <memory>
+#include <vector>
+
+#include "gtest/gtest.h"
+
 #include <rclcpp/exceptions.hpp>
 #include <rclcpp/node.hpp>
 #include <rclcpp/rclcpp.hpp>
 #include <rcpputils/scope_exit.hpp>
 #include <test_msgs/action/fibonacci.hpp>
 
-#include <gtest/gtest.h>
-
-#include <memory>
-#include <vector>
-
 #include "rcl_action/action_server.h"
 #include "rcl_action/wait.h"
 #include "rclcpp_action/create_server.hpp"
 #include "rclcpp_action/server.hpp"
-#include "./mocking_utils/patch.hpp"
+#include "mocking_utils/patch.hpp"
 
 using Fibonacci = test_msgs::action::Fibonacci;
 using CancelResponse = typename Fibonacci::Impl::CancelGoalService::Response;

diff --git a/rclcpp_action/test/test_server_goal_handle.cpp b/rclcpp_action/test/test_server_goal_handle.cpp
@@ -12,19 +12,19 @@
 // See the License for the specific language governing permissions and
 // limitations under the License.
 
+#include <memory>
+#include <vector>
+
+#include "gtest/gtest.h"
+
 #include <rclcpp/exceptions.hpp>
 #include <rclcpp/node.hpp>
 #include <rclcpp/rclcpp.hpp>
 #include <test_msgs/action/fibonacci.hpp>
 
-#include <gtest/gtest.h>
-
-#include <memory>
-#include <vector>
-
 #include "action_msgs/msg/goal_info.h"
 #include "rclcpp_action/server_goal_handle.hpp"
-#include "./mocking_utils/patch.hpp"
+#include "mocking_utils/patch.hpp"
 
 class FibonacciServerGoalHandle
   : public rclcpp_action::ServerGoalHandle<test_msgs::action::Fibonacci>