Mirrors/servo
1
0
Fork 0
mirror of https://github.com/servo/servo.git synced 2025-08-04 13:10:20 +01:00
Code Issues Projects Releases Packages Wiki Activity

Use append instead of extend where possible

Browse source 
The new `Vec::append` method is clearer and potentially faster.
This commit is contained in:
Matt Brubeck 2015-03-18 17:17:33 -07:00
parent 6ef9cb4e18
commit 94a7563aea
2 changed files with 4 additions and 4 deletions
Download patch file Download diff file Expand all files Collapse all files

4
components/layout/inline.rs
View file

@ -743,8 +743,8 @@ impl InlineFragments {
} }
/// Merges another set of inline fragments with this one. /// Merges another set of inline fragments with this one.
pub fn push_all(&mut self, fragments: InlineFragments) { pub fn push_all(&mut self, mut other: InlineFragments) {
self.fragments.extend(fragments.fragments.into_iter()); self.fragments.append(&mut other.fragments);
} }
/// A convenience function to return the fragment at a given index. /// A convenience function to return the fragment at a given index.

4
tests/reftest.rs
View file

@ -73,9 +73,9 @@ fn main() {
match maybe_extension { match maybe_extension {
Some(extension) => { Some(extension) => {
if extension.to_ascii_lowercase().as_slice() == "list" && file.is_file() { if extension.to_ascii_lowercase().as_slice() == "list" && file.is_file() {
let tests = parse_lists(&file, servo_args, render_mode, all_tests.len()); let mut tests = parse_lists(&file, servo_args, render_mode, all_tests.len());
println!("\t{} [{} tests]", file.display(), tests.len()); println!("\t{} [{} tests]", file.display(), tests.len());
all_tests.extend(tests.into_iter()); all_tests.append(&mut tests);
} }
} }
_ => {} _ => {}