diff --git a/servo/src/components/script/dom/bindings/trace.rs b/servo/src/components/script/dom/bindings/trace.rs
index cbbeddb85948..129eafedfe7d 100644
--- a/servo/src/components/script/dom/bindings/trace.rs
+++ b/servo/src/components/script/dom/bindings/trace.rs
@@ -138,8 +138,7 @@ impl<T> DerefMut<T> for Traceable<T> {
 
 impl<S: Encoder<E>, E, T: Encodable<S, E>> Encodable<S, E> for Traceable<RefCell<T>> {
     fn encode(&self, s: &mut S) -> Result<(), E> {
-        self.borrow().encode(s);
-        Ok(())
+        self.borrow().encode(s)
     }
 }
 
diff --git a/servo/src/components/script/dom/domimplementation.rs b/servo/src/components/script/dom/domimplementation.rs
index 725f5a12481b..a21a20f56345 100644
--- a/servo/src/components/script/dom/domimplementation.rs
+++ b/servo/src/components/script/dom/domimplementation.rs
@@ -82,7 +82,7 @@ impl<'a> DOMImplementationMethods for JSRef<'a, DOMImplementation> {
         // Step 1.
         let mut doc = Document::new(&win.root_ref(), None, NonHTMLDocument, None).root();
         // Step 2-3.
-        let mut maybe_elem = if qname.is_empty() {
+        let maybe_elem = if qname.is_empty() {
             None
         } else {
             match doc.CreateElementNS(namespace, qname) {
diff --git a/servo/src/components/script/layout_interface.rs b/servo/src/components/script/layout_interface.rs
index 876d37f43af8..b971d2ee2dd0 100644
--- a/servo/src/components/script/layout_interface.rs
+++ b/servo/src/components/script/layout_interface.rs
@@ -73,8 +73,7 @@ impl<S: Encoder<E>, E> Encodable<S, E> for TrustedNodeAddress {
         let node = addr as *Node as *mut Node;
         unsafe {
             JS::from_raw(node).encode(s)
-        };
-        Ok(())
+        }
     }
 }