diff --git a/servo/components/layout/inline.rs b/servo/components/layout/inline.rs index 4e634d36101b2..93a3080126e32 100644 --- a/servo/components/layout/inline.rs +++ b/servo/components/layout/inline.rs @@ -962,13 +962,29 @@ impl InlineFlow { Some(ref runs) => runs[run_count - run_idx - 1], None => (line.range, bidi::Level::ltr()) }; + + struct MaybeReverse { + iter: I, + reverse: bool, + } + + impl Iterator for MaybeReverse { + type Item = I::Item; + + fn next(&mut self) -> Option { + if self.reverse { + self.iter.next_back() + } else { + self.iter.next() + } + } + } + - let reverse = level.is_ltr() != is_ltr; - let fragment_indices = if reverse { - (range.end().get() - 1..range.begin().get() - 1).step_by(-1) - } else { - (range.begin().get()..range.end().get()).step_by(1) + let fragment_indices = MaybeReverse { + iter: range.begin().get()..range.end().get(), + reverse: level.is_ltr() != is_ltr, }; for fragment_index in fragment_indices { diff --git a/servo/components/layout/lib.rs b/servo/components/layout/lib.rs index 465c8d883f2a5..07bce47d559b6 100644 --- a/servo/components/layout/lib.rs +++ b/servo/components/layout/lib.rs @@ -8,7 +8,6 @@ #![feature(conservative_impl_trait)] #![feature(nonzero)] #![feature(raw)] -#![feature(step_by)] extern crate app_units; extern crate atomic_refcell; diff --git a/servo/components/selectors/parser.rs b/servo/components/selectors/parser.rs index dfa408855a969..65abfcd537e94 100644 --- a/servo/components/selectors/parser.rs +++ b/servo/components/selectors/parser.rs @@ -1572,7 +1572,6 @@ pub mod tests { use builder::HAS_PSEUDO_BIT; use cssparser::{Parser as CssParser, ToCss, serialize_identifier, ParserInput}; use parser; - use std::borrow::Cow; use std::collections::HashMap; use std::fmt; use super::*;