Use Option<T> to return from getters

This removes the cumbersome &mut bool argument and offers overall
a more readable code.

1 files changed, 4 insertions, 8 deletions

diff --git a/components/script/dom/namednodemap.rs b/components/script/dom/namednodemap.rs
index 10b6b8982ae..9edc1b1e93b 100644
--- a/components/script/dom/namednodemap.rs
+++ b/components/script/dom/namednodemap.rs
@@ -85,17 +85,13 @@ impl NamedNodeMapMethods for NamedNodeMap {
     }
 
     // https://dom.spec.whatwg.org/#dom-namednodemap-item
-    fn IndexedGetter(&self, index: u32, found: &mut bool) -> Option<Root<Attr>> {
-        let item = self.Item(index);
-        *found = item.is_some();
-        item
+    fn IndexedGetter(&self, index: u32) -> Option<Root<Attr>> {
+        self.Item(index)
     }
 
     // check-tidy: no specs after this line
-    fn NamedGetter(&self, name: DOMString, found: &mut bool) -> Option<Root<Attr>> {
-        let item = self.GetNamedItem(name);
-        *found = item.is_some();
-        item
+    fn NamedGetter(&self, name: DOMString) -> Option<Root<Attr>> {
+        self.GetNamedItem(name)
     }
 
     // https://heycam.github.io/webidl/#dfn-supported-property-names