review fixup: use while loop

Signed-off-by: Harald Hoyer <harald@profian.com>
tokio-rs · Mar 1, 2022 · b959099 · b959099
1 parent c956a1a
commit b959099
Showing 1 changed file with 7 additions and 12 deletions.
diff --git a/src/sys/wasi/mod.rs b/src/sys/wasi/mod.rs
@@ -180,20 +180,15 @@ impl Selector {
                 _ => false,
             }
         };
-        if let Some(index) = subscriptions.iter().position(predicate) {
+
+        let mut ret = Err(io::ErrorKind::NotFound.into());
+
+        while let Some(index) = subscriptions.iter().position(predicate) {
             subscriptions.swap_remove(index);
-            // We might have two subscriptions (read and write).
-            loop {
-                if let Some(index) = subscriptions.iter().position(predicate) {
-                    subscriptions.swap_remove(index);
-                } else {
-                    break;
-                }
-            }
-            Ok(())
-        } else {
-            Err(io::ErrorKind::NotFound.into())
+            ret = Ok(())
         }
+
+        ret
     }
 }