From e246836798e8f0f7c17356ed848e296d61365adc Mon Sep 17 00:00:00 2001 From: David Tolnay Date: Fri, 12 Apr 2019 19:58:34 -0700 Subject: clippy: Resolve while_let_loop TEST=bin/clippy Change-Id: Iae447ec2c1b095286ab02cd153bd2daafe95fae1 Reviewed-on: https://chromium-review.googlesource.com/1566894 Commit-Ready: David Tolnay Tested-by: David Tolnay Tested-by: kokoro Reviewed-by: Daniel Verkamp --- devices/src/virtio/gpu/mod.rs | 22 ++++++---------------- 1 file changed, 6 insertions(+), 16 deletions(-) (limited to 'devices/src/virtio/gpu') diff --git a/devices/src/virtio/gpu/mod.rs b/devices/src/virtio/gpu/mod.rs index 251b59c..686c174 100644 --- a/devices/src/virtio/gpu/mod.rs +++ b/devices/src/virtio/gpu/mod.rs @@ -570,26 +570,16 @@ impl Worker { } // All cursor commands go first because they have higher priority. - loop { - match self.state.process_cursor(&self.mem) { - Some(ReturnDescriptor { index, len }) => { - self.cursor_queue.add_used(&self.mem, index, len); - signal_used = true; - } - None => break, - } + while let Some(desc) = self.state.process_cursor(&self.mem) { + self.cursor_queue.add_used(&self.mem, desc.index, desc.len); + signal_used = true; } self.state.fence_poll(); - loop { - match self.state.process_ctrl(&self.mem) { - Some(ReturnDescriptor { index, len }) => { - self.ctrl_queue.add_used(&self.mem, index, len); - signal_used = true; - } - None => break, - } + while let Some(desc) = self.state.process_ctrl(&self.mem) { + self.ctrl_queue.add_used(&self.mem, desc.index, desc.len); + signal_used = true; } // Process the entire control queue before the resource bridge in case a resource is -- cgit 1.4.1