From 39aa3cfc2ab7e3266d725556df4f4ca7c1bf61b1 Mon Sep 17 00:00:00 2001 From: Raphael Nestler Date: Sat, 7 Nov 2015 16:21:33 +0100 Subject: [PATCH] Use while let{} instead of loop{match{}} in ScrollingTimer::run() --- components/compositing/scrolling.rs | 15 +++++---------- 1 file changed, 5 insertions(+), 10 deletions(-) diff --git a/components/compositing/scrolling.rs b/components/compositing/scrolling.rs index df266f1741e6..c4f4be7804c6 100644 --- a/components/compositing/scrolling.rs +++ b/components/compositing/scrolling.rs @@ -54,16 +54,11 @@ impl ScrollingTimerProxy { impl ScrollingTimer { pub fn run(&mut self) { - loop { - match self.receiver.recv() { - Ok(ToScrollingTimerMsg::ScrollEventProcessedMsg(timestamp)) => { - let target = timestamp as i64 + TIMEOUT; - let delta_ns = target - (time::precise_time_ns() as i64); - sleep_ms((delta_ns / 1000000) as u32); - self.compositor_proxy.send(Msg::ScrollTimeout(timestamp)); - } - Ok(ToScrollingTimerMsg::ExitMsg) | Err(_) => break, - } + while let Ok(ToScrollingTimerMsg::ScrollEventProcessedMsg(timestamp)) = self.receiver.recv() { + let target = timestamp as i64 + TIMEOUT; + let delta_ns = target - (time::precise_time_ns() as i64); + sleep_ms((delta_ns / 1000000) as u32); + self.compositor_proxy.send(Msg::ScrollTimeout(timestamp)); } } }