Auto merge of #8393 - rnestler:fix_8384, r=frewsxcv

Use while let{} instead of loop{match{}}

This fixes #8384

<!-- Reviewable:start -->
[<img src="https://reviewable.io/review_button.png" height=40 alt="Review on Reviewable"/>](https://reviewable.io/reviews/servo/servo/8393)
<!-- Reviewable:end -->
This commit is contained in:
bors-servo 2015-11-08 01:46:21 +05:30
commit 1979d0a2e6
2 changed files with 8 additions and 18 deletions

View file

@ -2125,14 +2125,9 @@ fn find_layer_with_pipeline_and_layer_id_for_layer(layer: Rc<Layer<CompositorDat
impl<Window> CompositorEventListener for IOCompositor<Window> where Window: WindowMethods { impl<Window> CompositorEventListener for IOCompositor<Window> where Window: WindowMethods {
fn handle_events(&mut self, messages: Vec<WindowEvent>) -> bool { fn handle_events(&mut self, messages: Vec<WindowEvent>) -> bool {
// Check for new messages coming from the other tasks in the system. // Check for new messages coming from the other tasks in the system.
loop { while let Some(msg) = self.port.try_recv_compositor_msg() {
match self.port.try_recv_compositor_msg() { if !self.handle_browser_message(msg) {
None => break, break
Some(msg) => {
if !self.handle_browser_message(msg) {
break
}
}
} }
} }

View file

@ -54,16 +54,11 @@ impl ScrollingTimerProxy {
impl ScrollingTimer { impl ScrollingTimer {
pub fn run(&mut self) { pub fn run(&mut self) {
loop { while let Ok(ToScrollingTimerMsg::ScrollEventProcessedMsg(timestamp)) = self.receiver.recv() {
match self.receiver.recv() { let target = timestamp as i64 + TIMEOUT;
Ok(ToScrollingTimerMsg::ScrollEventProcessedMsg(timestamp)) => { let delta_ns = target - (time::precise_time_ns() as i64);
let target = timestamp as i64 + TIMEOUT; sleep_ms((delta_ns / 1000000) as u32);
let delta_ns = target - (time::precise_time_ns() as i64); self.compositor_proxy.send(Msg::ScrollTimeout(timestamp));
sleep_ms((delta_ns / 1000000) as u32);
self.compositor_proxy.send(Msg::ScrollTimeout(timestamp));
}
Ok(ToScrollingTimerMsg::ExitMsg) | Err(_) => break,
}
} }
} }
} }