Break line after paren in multiline function calls

rotu · rotu · commit 642d349da02c · 2019-07-16T17:57:16.000-05:00
as per ament/ament_lint#148 Signed-off-by: Dan Rose <dan@digilabs.io>
diff --git a/composition/src/server_component.cpp b/composition/src/server_component.cpp
@@ -33,7 +33,8 @@ Server::Server(const rclcpp::NodeOptions & options)
     std::shared_ptr<example_interfaces::srv::AddTwoInts::Response> response
     ) -> void
     {
-      RCLCPP_INFO(this->get_logger(), "Incoming request: [a: %" PRId64 ", b: %" PRId64 "]",
+      RCLCPP_INFO(
+        this->get_logger(), "Incoming request: [a: %" PRId64 ", b: %" PRId64 "]",
         request->a, request->b);
       std::flush(std::cout);
       response->sum = request->a + request->b;
diff --git a/demo_nodes_cpp/src/parameters/even_parameters_node.cpp b/demo_nodes_cpp/src/parameters/even_parameters_node.cpp
@@ -43,30 +43,34 @@ class EvenParameterNode : public rclcpp::Node
         for (auto parameter : parameters) {
           rclcpp::ParameterType parameter_type = parameter.get_type();
           if (rclcpp::ParameterType::PARAMETER_NOT_SET == parameter_type) {
-            RCLCPP_INFO(this->get_logger(),
+            RCLCPP_INFO(
+              this->get_logger(),
               "parameter '%s' deleted successfully",
               parameter.get_name().c_str()
             );
             result.successful &= true;
           } else if (rclcpp::ParameterType::PARAMETER_INTEGER == parameter_type) {
             if (parameter.as_int() % 2 != 0) {
-              RCLCPP_INFO(this->get_logger(),
+              RCLCPP_INFO(
+                this->get_logger(),
                 "Requested value '%d' for parameter '%s' is not an even number:"
                 " rejecting change...",
                 parameter.as_int(),
                 parameter.get_name().c_str()
               );
               result.successful = false;
             } else {
-              RCLCPP_INFO(this->get_logger(),
+              RCLCPP_INFO(
+                this->get_logger(),
                 "parameter '%s' has changed and is now: %s",
                 parameter.get_name().c_str(),
                 parameter.value_to_string().c_str()
               );
               result.successful &= true;
             }
           } else {
-            RCLCPP_INFO(this->get_logger(),
+            RCLCPP_INFO(
+              this->get_logger(),
               "only integer parameters can be set\n"
               "requested value for parameter '%s' is not an even number, rejecting change...",
               parameter.get_name().c_str()
diff --git a/demo_nodes_cpp/src/parameters/list_parameters.cpp b/demo_nodes_cpp/src/parameters/list_parameters.cpp
@@ -47,7 +47,8 @@ int main(int argc, char ** argv)
 
   RCLCPP_INFO(node->get_logger(), "Setting parameters...");
   // Set several different types of parameters.
-  auto set_parameters_results = parameters_client->set_parameters({
+  auto set_parameters_results = parameters_client->set_parameters(
+  {
     rclcpp::Parameter("foo", 2),
     rclcpp::Parameter("bar", "hello"),
     rclcpp::Parameter("baz", 1.45),
diff --git a/demo_nodes_cpp/src/parameters/list_parameters_async.cpp b/demo_nodes_cpp/src/parameters/list_parameters_async.cpp
@@ -48,7 +48,8 @@ int main(int argc, char ** argv)
 
   RCLCPP_INFO(node->get_logger(), "Setting parameters...");
   // Set several differnet types of parameters.
-  auto results = parameters_client->set_parameters({
+  auto results = parameters_client->set_parameters(
+  {
     rclcpp::Parameter("foo", 2),
     rclcpp::Parameter("bar", "hello"),
     rclcpp::Parameter("baz", 1.45),
diff --git a/demo_nodes_cpp/src/parameters/parameter_blackboard.cpp b/demo_nodes_cpp/src/parameters/parameter_blackboard.cpp
@@ -31,7 +31,8 @@ class ParameterBlackboard : public rclcpp::Node
   ))
   : rclcpp::Node(name, namespace_, options)
   {
-    RCLCPP_INFO(this->get_logger(),
+    RCLCPP_INFO(
+      this->get_logger(),
       "Parameter blackboard node named '%s' ready, and serving '%zu' parameters already!",
       this->get_fully_qualified_name(), this->list_parameters(
         {}, rcl_interfaces::srv::ListParameters::Request::DEPTH_RECURSIVE).names.size());
diff --git a/demo_nodes_cpp/src/parameters/parameter_events.cpp b/demo_nodes_cpp/src/parameters/parameter_events.cpp
@@ -73,15 +73,17 @@ int main(int argc, char ** argv)
   node->declare_parameter("foobar");
 
   // Set several different types of parameters.
-  auto set_parameters_results = parameters_client->set_parameters({
+  auto set_parameters_results = parameters_client->set_parameters(
+  {
     rclcpp::Parameter("foo", 2),
     rclcpp::Parameter("bar", "hello"),
     rclcpp::Parameter("baz", 1.45),
     rclcpp::Parameter("foobar", true),
   });
 
   // Change the value of some of them.
-  set_parameters_results = parameters_client->set_parameters({
+  set_parameters_results = parameters_client->set_parameters(
+  {
     rclcpp::Parameter("foo", 3),
     rclcpp::Parameter("bar", "world"),
   });
diff --git a/demo_nodes_cpp/src/parameters/parameter_events_async.cpp b/demo_nodes_cpp/src/parameters/parameter_events_async.cpp
@@ -65,10 +65,11 @@ class ParameterEventsAsyncNode : public rclcpp::Node
 
     // Queue a `set_parameters` request as soon as `spin` is called on this node.
     // TODO(dhood): consider adding a "call soon" notion to Node to not require a timer for this.
-    timer_ = create_wall_timer(200ms,
-        [this]() {
-          this->queue_first_set_parameter_request();
-        });
+    timer_ = create_wall_timer(
+      200ms,
+      [this]() {
+        this->queue_first_set_parameter_request();
+      });
   }
 
   // Set several different types of parameters.
@@ -93,7 +94,8 @@ class ParameterEventsAsyncNode : public rclcpp::Node
         this->queue_second_set_parameter_request();
       };
 
-    parameters_client_->set_parameters({
+    parameters_client_->set_parameters(
+    {
       rclcpp::Parameter("foo", 2),
       rclcpp::Parameter("bar", "hello"),
       rclcpp::Parameter("baz", 1.45),
@@ -114,12 +116,14 @@ class ParameterEventsAsyncNode : public rclcpp::Node
         // TODO(wjwwood): Create and use delete_parameter
 
         // Give time for all of the ParameterEvent callbacks to be received.
-        timer_ = create_wall_timer(100ms,
-            []() {
-              rclcpp::shutdown();
-            });
+        timer_ = create_wall_timer(
+          100ms,
+          []() {
+            rclcpp::shutdown();
+          });
       };
-    parameters_client_->set_parameters({
+    parameters_client_->set_parameters(
+    {
       rclcpp::Parameter("foo", 3),
       rclcpp::Parameter("bar", "world"),
     }, response_received_callback);
diff --git a/demo_nodes_cpp/src/parameters/set_and_get_parameters.cpp b/demo_nodes_cpp/src/parameters/set_and_get_parameters.cpp
@@ -47,7 +47,8 @@ int main(int argc, char ** argv)
   }
 
   // Set several different types of parameters.
-  auto set_parameters_results = parameters_client->set_parameters({
+  auto set_parameters_results = parameters_client->set_parameters(
+  {
     rclcpp::Parameter("foo", 2),
     rclcpp::Parameter("bar", "hello"),
     rclcpp::Parameter("baz", 1.45),
diff --git a/demo_nodes_cpp/src/parameters/set_and_get_parameters_async.cpp b/demo_nodes_cpp/src/parameters/set_and_get_parameters_async.cpp
@@ -47,7 +47,8 @@ int main(int argc, char ** argv)
   }
 
   // Set several different types of parameters.
-  auto results = parameters_client->set_parameters({
+  auto results = parameters_client->set_parameters(
+  {
     rclcpp::Parameter("foo", 2),
     rclcpp::Parameter("bar", "hello"),
     rclcpp::Parameter("baz", 1.45),
diff --git a/demo_nodes_cpp/src/services/add_two_ints_server.cpp b/demo_nodes_cpp/src/services/add_two_ints_server.cpp
@@ -45,7 +45,8 @@ class ServerNode : public rclcpp::Node
         std::shared_ptr<example_interfaces::srv::AddTwoInts::Response> response) -> void
       {
         (void)request_header;
-        RCLCPP_INFO(this->get_logger(), "Incoming request\na: %" PRId64 " b: %" PRId64,
+        RCLCPP_INFO(
+          this->get_logger(), "Incoming request\na: %" PRId64 " b: %" PRId64,
           request->a, request->b);
         response->sum = request->a + request->b;
       };
diff --git a/demo_nodes_cpp/src/timers/one_off_timer.cpp b/demo_nodes_cpp/src/timers/one_off_timer.cpp
@@ -32,10 +32,11 @@ class OneOffTimerNode : public rclcpp::Node
         RCLCPP_INFO(this->get_logger(), "in periodic_timer callback");
         if (this->count++ % 3 == 0) {
           RCLCPP_INFO(this->get_logger(), "  resetting one off timer");
-          this->one_off_timer = this->create_wall_timer(1s, [this]() {
-            RCLCPP_INFO(this->get_logger(), "in one_off_timer callback");
-            this->one_off_timer->cancel();
-          });
+          this->one_off_timer = this->create_wall_timer(
+            1s, [this]() {
+              RCLCPP_INFO(this->get_logger(), "in one_off_timer callback");
+              this->one_off_timer->cancel();
+            });
         } else {
           RCLCPP_INFO(this->get_logger(), "  not resetting one off timer");
         }
diff --git a/image_tools/src/burger.cpp b/image_tools/src/burger.cpp
@@ -126,12 +126,14 @@ cv::Mat & Burger::render_burger(size_t width, size_t height)
   }
   burger_buf = cv::Scalar(0, 0, 0);
   for (int b = 0; b < static_cast<int>(x.size()); b++) {
-    burger_template.copyTo(burger_buf(cv::Rect(
-        x[b],
-        y[b],
-        burger_template.size().height,
-        burger_template.size().width
-      )), burger_mask);
+    burger_template.copyTo(
+      burger_buf(
+        cv::Rect(
+          x[b],
+          y[b],
+          burger_template.size().height,
+          burger_template.size().width
+        )), burger_mask);
     x[b] += x_inc[b];
     y[b] += y_inc[b];
     // bounce as needed
diff --git a/lifecycle/src/lifecycle_listener.cpp b/lifecycle/src/lifecycle_listener.cpp
@@ -38,8 +38,9 @@ class LifecycleListener : public rclcpp::Node
   : Node(node_name)
   {
     // Data topic from the lc_talker node
-    sub_data_ = this->create_subscription<std_msgs::msg::String>("lifecycle_chatter", 10,
-        std::bind(&LifecycleListener::data_callback, this, std::placeholders::_1));
+    sub_data_ = this->create_subscription<std_msgs::msg::String>(
+      "lifecycle_chatter", 10,
+      std::bind(&LifecycleListener::data_callback, this, std::placeholders::_1));
 
     // Notification event topic. All state changes
     // are published here as TransitionEvents with
@@ -57,7 +58,8 @@ class LifecycleListener : public rclcpp::Node
 
   void notification_callback(const lifecycle_msgs::msg::TransitionEvent::SharedPtr msg)
   {
-    RCLCPP_INFO(get_logger(), "notify callback: Transition from state %s to %s",
+    RCLCPP_INFO(
+      get_logger(), "notify callback: Transition from state %s to %s",
       msg->start_state.label.c_str(), msg->goal_state.label.c_str());
   }
 
diff --git a/lifecycle/src/lifecycle_service_client.cpp b/lifecycle/src/lifecycle_service_client.cpp
@@ -120,7 +120,8 @@ class LifecycleServiceClient : public rclcpp::Node
 
     // We have an succesful answer. So let's print the current state.
     if (future_result.get()) {
-      RCLCPP_INFO(get_logger(), "Node %s has current state %s.",
+      RCLCPP_INFO(
+        get_logger(), "Node %s has current state %s.",
         lifecycle_node, future_result.get()->current_state.label.c_str());
       return future_result.get()->current_state.id;
     } else {
@@ -317,8 +318,9 @@ int main(int argc, char ** argv)
   rclcpp::executors::SingleThreadedExecutor exe;
   exe.add_node(lc_client);
 
-  std::shared_future<void> script = std::async(std::launch::async,
-      std::bind(callee_script, lc_client));
+  std::shared_future<void> script = std::async(
+    std::launch::async,
+    std::bind(callee_script, lc_client));
   exe.spin_until_future_complete(script);
 
   rclcpp::shutdown();
diff --git a/pendulum_control/include/pendulum_control/pendulum_motor.hpp b/pendulum_control/include/pendulum_control/pendulum_motor.hpp
@@ -82,7 +82,8 @@ class PendulumMotor
     thread_param.sched_priority = 90;
     pthread_attr_setschedparam(&thread_attr_, &thread_param);
     pthread_attr_setschedpolicy(&thread_attr_, SCHED_RR);
-    pthread_create(&physics_update_thread_, &thread_attr_,
+    pthread_create(
+      &physics_update_thread_, &thread_attr_,
       &pendulum_control::PendulumMotor::physics_update_wrapper, this);
   }
 
diff --git a/pendulum_control/src/pendulum_teleop.cpp b/pendulum_control/src/pendulum_teleop.cpp
@@ -34,7 +34,8 @@ int main(int argc, char * argv[])
 
   double command = M_PI / 2;
   if (argc < 2) {
-    fprintf(stderr,
+    fprintf(
+      stderr,
       "Command argument not specified. Setting command to 90 degrees (PI/2 radians).\n");
   } else {
     command = atof(argv[1]);
diff --git a/quality_of_service_demo/rclcpp/src/deadline.cpp b/quality_of_service_demo/rclcpp/src/deadline.cpp
@@ -34,23 +34,27 @@ static const size_t DEFAULT_PAUSE_FOR = 1000;
 void print_usage()
 {
   printf("Usage for deadline:\n");
-  printf("deadline "
+  printf(
+    "deadline "
     "deadline_duration "
     "[%s publish_for] "
     "[%s pause_for] "
     "[-h]\n",
     OPTION_PUBLISH_FOR,
     OPTION_PAUSE_FOR);
   printf("required arguments:\n");
-  printf("deadline_duration: "
+  printf(
+    "deadline_duration: "
     "Duration in positive integer milliseconds of the Deadline QoS setting.\n");
   printf("optional arguments:\n");
   printf("-h : Print this help message.\n");
-  printf("%s publish_for : "
+  printf(
+    "%s publish_for : "
     "Duration to publish (in positive integer milliseconds) until pausing the talker. "
     "Defaults to %zu.\n",
     OPTION_PUBLISH_FOR, DEFAULT_PUBLISH_FOR);
-  printf("%s pause_for : "
+  printf(
+    "%s pause_for : "
     "Duration to pause (in positive integer milliseconds) before starting to publish again. "
     "Defaults to %zu.\n",
     OPTION_PAUSE_FOR, DEFAULT_PAUSE_FOR);
@@ -94,7 +98,8 @@ int main(int argc, char * argv[])
   talker->get_options().event_callbacks.deadline_callback =
     [node = talker.get()](rclcpp::QOSDeadlineOfferedInfo & event) -> void
     {
-      RCLCPP_INFO(node->get_logger(),
+      RCLCPP_INFO(
+        node->get_logger(),
         "Offered deadline missed - total %d delta %d",
         event.total_count, event.total_count_change);
     };
@@ -103,7 +108,8 @@ int main(int argc, char * argv[])
   listener->get_options().event_callbacks.deadline_callback =
     [node = listener.get()](rclcpp::QOSDeadlineRequestedInfo & event) -> void
     {
-      RCLCPP_INFO(node->get_logger(),
+      RCLCPP_INFO(
+        node->get_logger(),
         "Requested deadline missed - total %d delta %d",
         event.total_count, event.total_count_change);
     };
diff --git a/quality_of_service_demo/rclcpp/src/interactive_publisher.cpp b/quality_of_service_demo/rclcpp/src/interactive_publisher.cpp
@@ -115,12 +115,14 @@ int main(int argc, char * argv[])
     return 0;
   }
   if (rcutils_cli_option_exist(argv, argv + argc, OPTION_PUBLISH_DELAY)) {
-    publish_delay = std::chrono::milliseconds(static_cast<int>(1000 *
-        std::stof(rcutils_cli_get_option(argv, argv + argc, OPTION_PUBLISH_DELAY))));
+    publish_delay = std::chrono::milliseconds(
+      static_cast<int>(1000 *
+      std::stof(rcutils_cli_get_option(argv, argv + argc, OPTION_PUBLISH_DELAY))));
   }
   if (rcutils_cli_option_exist(argv, argv + argc, OPTION_DEADLINE_PERIOD)) {
-    auto period = std::chrono::milliseconds(static_cast<int>(1000 *
-        std::stof(rcutils_cli_get_option(argv, argv + argc, OPTION_DEADLINE_PERIOD))));
+    auto period = std::chrono::milliseconds(
+      static_cast<int>(1000 *
+      std::stof(rcutils_cli_get_option(argv, argv + argc, OPTION_DEADLINE_PERIOD))));
     qos_settings.deadline(period);
   }
   if (rcutils_cli_option_exist(argv, argv + argc, OPTION_LIVELINESS_KIND)) {
@@ -138,20 +140,23 @@ int main(int argc, char * argv[])
     }
   }
   if (rcutils_cli_option_exist(argv, argv + argc, OPTION_LEASE_DURATION)) {
-    auto duration = std::chrono::milliseconds(static_cast<int>(1000 *
-        std::stof(rcutils_cli_get_option(argv, argv + argc, OPTION_LEASE_DURATION))));
+    auto duration = std::chrono::milliseconds(
+      static_cast<int>(1000 *
+      std::stof(rcutils_cli_get_option(argv, argv + argc, OPTION_LEASE_DURATION))));
     qos_settings.liveliness_lease_duration(duration);
   }
 
   auto talker = std::make_shared<Talker>(qos_settings, DEFAULT_TOPIC_NAME, 0, publish_delay);
   talker->get_options().event_callbacks.deadline_callback =
     [node = talker.get()](rclcpp::QOSDeadlineOfferedInfo & event) {
-      RCLCPP_INFO(node->get_logger(), "Deadline missed - total %d (delta %d)",
+      RCLCPP_INFO(
+        node->get_logger(), "Deadline missed - total %d (delta %d)",
         event.total_count, event.total_count_change);
     };
   talker->get_options().event_callbacks.liveliness_callback =
     [node = talker.get()](rclcpp::QOSLivelinessLostInfo & event) {
-      RCLCPP_INFO(node->get_logger(), "Liveliness lost - total %d (delta %d)",
+      RCLCPP_INFO(
+        node->get_logger(), "Liveliness lost - total %d (delta %d)",
         event.total_count, event.total_count_change);
     };
 
diff --git a/quality_of_service_demo/rclcpp/src/interactive_subscriber.cpp b/quality_of_service_demo/rclcpp/src/interactive_subscriber.cpp
diff --git a/quality_of_service_demo/rclcpp/src/lifespan.cpp b/quality_of_service_demo/rclcpp/src/lifespan.cpp
diff --git a/quality_of_service_demo/rclcpp/src/liveliness.cpp b/quality_of_service_demo/rclcpp/src/liveliness.cpp

Original file line number	Diff line number	Diff line change
`@@ -47,7 +47,8 @@ int main(int argc, char ** argv)`
`47`	`47`	`}`
`48`	`48`
`49`	`49`	`// Set several different types of parameters.`
`50`		`- auto set_parameters_results = parameters_client->set_parameters({`
	`50`	`+ auto set_parameters_results = parameters_client->set_parameters(`
	`51`	`+ {`
`51`	`52`	`rclcpp::Parameter("foo", 2),`
`52`	`53`	`rclcpp::Parameter("bar", "hello"),`
`53`	`54`	`rclcpp::Parameter("baz", 1.45),`