From de6c78c21eb0b283853c087410e21eb1dfffa643 Mon Sep 17 00:00:00 2001
From: Mark Rousskov <mark.simulacrum@gmail.com>
Date: Sun, 26 Jan 2020 12:16:01 -0500
Subject: [PATCH] Polish code to clarify meaning

This preserves the same behavior, but the previous code looked like it might
loop, but in fact never did.
---
 src/cargo/core/compiler/context/mod.rs | 14 ++++++--------
 1 file changed, 6 insertions(+), 8 deletions(-)

diff --git a/src/cargo/core/compiler/context/mod.rs b/src/cargo/core/compiler/context/mod.rs
index 6adccbd4cd4..8df9d799ccd 100644
--- a/src/cargo/core/compiler/context/mod.rs
+++ b/src/cargo/core/compiler/context/mod.rs
@@ -504,14 +504,12 @@ impl<'a, 'cfg> Context<'a, 'cfg> {
 
         // Drain the client fully
         for i in 0..tokens {
-            while let Err(e) = client.acquire_raw() {
-                anyhow::bail!(
-                    "failed to fully drain {}/{} token from jobserver at startup: {:?}",
-                    i,
-                    tokens,
-                    e,
-                );
-            }
+            client.acquire_raw().chain_err(|| {
+                format!(
+                    "failed to fully drain {}/{} token from jobserver at startup",
+                    i, tokens,
+                )
+            })?;
         }
 
         Ok(client)