remove expression that always evaluates to True

ros2 · Oct 18, 2017 · 6c77285 · 6c77285
1 parent 385b18b
commit 6c77285
Showing 1 changed file with 4 additions and 5 deletions.
diff --git a/rclpy/rclpy/executors.py b/rclpy/rclpy/executors.py
@@ -55,11 +55,10 @@ def wait(self, timeout_sec=None):
         if timeout_sec is not None and timeout_sec < 0:
             timeout_sec = None
         # Wait for all work to complete
-        if timeout_sec is None or timeout_sec >= 0:
-            with self._work_condition:
-                if not self._work_condition.wait_for(
-                        lambda: self._num_work_executing == 0, timeout_sec):
-                    return False
+        with self._work_condition:
+            if not self._work_condition.wait_for(
+                    lambda: self._num_work_executing == 0, timeout_sec):
+                return False
         return True