Fix feedback data race

// See ros2#2451 (comment)

Author: Mauro Passerino

rclcpp_action/src/client.cpp

@@ -355,16 +355,6 @@ ClientBase::is_ready(rcl_wait_set_t * wait_set)
 
   pimpl_->next_ready_event = ClientBaseImpl::NO_EVENT_READY;
 
-  if (is_feedback_ready) {
-    pimpl_->next_ready_event = static_cast<size_t>(EntityType::FeedbackSubscription);
-    return true;
-  }
-
-  if (is_status_ready) {
-    pimpl_->next_ready_event = static_cast<size_t>(EntityType::StatusSubscription);
-    return true;
-  }
-
   if (is_goal_response_ready) {
     pimpl_->next_ready_event = static_cast<size_t>(EntityType::GoalClient);
     return true;
@@ -380,6 +370,16 @@ ClientBase::is_ready(rcl_wait_set_t * wait_set)
     return true;
   }
 
+  if (is_feedback_ready) {
+    pimpl_->next_ready_event = static_cast<size_t>(EntityType::FeedbackSubscription);
+    return true;
+  }
+
+  if (is_status_ready) {
+    pimpl_->next_ready_event = static_cast<size_t>(EntityType::StatusSubscription);
+    return true;
+  }
+
   return false;
 }