diff options
| author | Anthony Ramine <n.oxyde@gmail.com> | 2016-08-29 00:55:29 +0200 |
|---|---|---|
| committer | Anthony Ramine <n.oxyde@gmail.com> | 2016-08-30 19:07:19 +0200 |
| commit | 7dfb336be8dae1e2be9b898c374b6715e2a00ac7 (patch) | |
| tree | 775fbf5f78d812db39ddfab79c56731262d5aded /components/script/dom/domstringmap.rs | |
| parent | 6e1523f4ae61c16578a462c2e5335cbc95a6ef04 (diff) | |
| download | servo-7dfb336be8dae1e2be9b898c374b6715e2a00ac7.tar.gz servo-7dfb336be8dae1e2be9b898c374b6715e2a00ac7.zip | |
Use Option<T> to return from getters
This removes the cumbersome &mut bool argument and offers overall
a more readable code.
Diffstat (limited to 'components/script/dom/domstringmap.rs')
| -rw-r--r-- | components/script/dom/domstringmap.rs | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/components/script/dom/domstringmap.rs b/components/script/dom/domstringmap.rs index c5e534d242e..11479d6b5df 100644 --- a/components/script/dom/domstringmap.rs +++ b/components/script/dom/domstringmap.rs @@ -47,10 +47,8 @@ impl DOMStringMapMethods for DOMStringMap { } // https://html.spec.whatwg.org/multipage/#dom-domstringmap-nameditem - fn NamedGetter(&self, name: DOMString, found: &mut bool) -> DOMString { - let attr = self.element.get_custom_attr(name); - *found = attr.is_some(); - attr.unwrap_or_default() + fn NamedGetter(&self, name: DOMString) -> Option<DOMString> { + self.element.get_custom_attr(name) } // https://html.spec.whatwg.org/multipage/#the-domstringmap-interface:supported-property-names |