---

yaml
---
r: 67221
b: refs/heads/master
c: 9a093ab
h: refs/heads/master
i:
  67219: eef6dfb
v: v3

Author: graydon

[refs]

@@ -1,5 +1,5 @@
 ---
-refs/heads/master: d5803e67eecda097932424a6c7b52cd1addd76b9
+refs/heads/master: 9a093ab70a7d7b12e9cddf644e1a975375937465
 refs/heads/snap-stage1: e33de59e47c5076a89eadeb38f4934f58a3618a6
 refs/heads/snap-stage3: 18e3db7392d2d0697b7e27d6d986139960144d85
 refs/heads/try: 7b78b52e602bb3ea8174f9b2006bff3315f03ef9

trunk/src/libextra/workcache.rs

@@ -330,37 +330,29 @@ impl<T:Send +
     pub fn new(p: @mut Prep, e: Either<T,PortOne<(Exec,T)>>) -> Work<T> {
         Work { prep: p, res: Some(e) }
     }
-}
 
-// FIXME (#3724): movable self. This should be in impl Work.
-fn unwrap<T:Send +
-            Encodable<json::Encoder> +
-            Decodable<json::Decoder>>( // FIXME(#5121)
-        w: Work<T>) -> T {
-    let mut ww = w;
-    let s = ww.res.take();
-
-    match s {
-        None => fail!(),
-        Some(Left(v)) => v,
-        Some(Right(port)) => {
-            let (exe, v) = recv_one(port);
-
-            let s = json_encode(&v);
-
-            let p = &*ww.prep;
-            do p.ctxt.db.write |db| {
-                db.cache(p.fn_name,
-                         &p.declared_inputs,
-                         &exe.discovered_inputs,
-                         &exe.discovered_outputs,
-                         s);
+    pub fn unwrap(self) -> T {
+        let Work { prep, res } = self;
+        match res {
+            None => fail!(),
+            Some(Left(v)) => v,
+            Some(Right(port)) => {
+                let (exe, v) = recv_one(port);
+                let s = json_encode(&v);
+                do prep.ctxt.db.write |db| {
+                    db.cache(prep.fn_name,
+                             &prep.declared_inputs,
+                             &exe.discovered_inputs,
+                             &exe.discovered_outputs,
+                             s);
+                }
+                v
             }
-            v
         }
     }
 }
 
+
 //#[test]
 fn test() {
     use std::io::WriterUtil;
@@ -385,6 +377,6 @@ fn test() {
             out.to_str()
         }
     };
-    let s = unwrap(w);
+    let s = w.unwrap();
     io::println(s);
 }