From bbe494acde06e6b36799435291a5136ebb7501e0 Mon Sep 17 00:00:00 2001 From: Ms2ger Date: Sat, 11 Apr 2015 21:30:56 +0200 Subject: [PATCH] Use if let remove_inline_style_property. It makes little sense to use map() in a case where we don't care about the return value. --- components/script/dom/element.rs | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/components/script/dom/element.rs b/components/script/dom/element.rs index 4625795b6c0..523caf59a92 100644 --- a/components/script/dom/element.rs +++ b/components/script/dom/element.rs @@ -507,7 +507,7 @@ impl<'a> ElementHelpers<'a> for JSRef<'a, Element> { fn remove_inline_style_property(self, property: DOMString) { let mut inline_declarations = self.style_attribute.borrow_mut(); - inline_declarations.as_mut().map(|declarations| { + if let &mut Some(ref mut declarations) = &mut *inline_declarations { let index = declarations.normal .iter() .position(|decl| decl.name() == property); @@ -523,7 +523,7 @@ impl<'a> ElementHelpers<'a> for JSRef<'a, Element> { declarations.important.make_unique().remove(index); return; } - }); + } } fn update_inline_style(self, property_decl: PropertyDeclaration, style_priority: StylePriority) {