Apply suggestions from code review

Co-authored-by: Bence Magyar <[email protected]>
Co-authored-by: Sai Kishor Kothakota <[email protected]>

Author: 3 people

joint_trajectory_controller/src/joint_trajectory_controller.cpp

@@ -334,7 +334,7 @@ controller_interface::return_type JointTrajectoryController::update(
         {
           auto result = std::make_shared<FollowJTrajAction::Result>();
           result->set__error_code(FollowJTrajAction::Result::PATH_TOLERANCE_VIOLATED);
-          result->set__error_string("Aborted due to state tolerance violation");
+          result->set__error_string("Aborted due to path tolerance violation");
           active_goal->setAborted(result);
           // TODO(matthew-reynolds): Need a lock-free write here
           // See https://github.com/ros-controls/ros2_controllers/issues/168
@@ -353,7 +353,7 @@ controller_interface::return_type JointTrajectoryController::update(
           {
             auto result = std::make_shared<FollowJTrajAction::Result>();
             result->set__error_code(FollowJTrajAction::Result::SUCCESSFUL);
-            result->set__error_string("Goal reached, success!");
+            result->set__error_string("Goal successfully reached!");
             active_goal->setSucceeded(result);
             // TODO(matthew-reynolds): Need a lock-free write here
             // See https://github.com/ros-controls/ros2_controllers/issues/168
@@ -367,7 +367,7 @@ controller_interface::return_type JointTrajectoryController::update(
           }
           else if (!within_goal_time)
           {
-            const std::string error_string = "Aborted due goal_time_tolerance exceeding by " +
+            const std::string error_string = "Aborted due to goal_time_tolerance exceeding by " +
                                              std::to_string(time_difference) + " seconds";
 
             auto result = std::make_shared<FollowJTrajAction::Result>();