rustdoc: Make doc::retdoc non-optional

diff --git a/src/rustdoc/attr_pass.rs b/src/rustdoc/attr_pass.rs
index 6d4cabba0d4f165e5dd5bdbe130a75233bc25939..4c945a7869f327312e1a34b58f2a74a9cee59476 100644 (file)
--- a/src/rustdoc/attr_pass.rs
+++ b/src/rustdoc/attr_pass.rs
@@ -158,20 +158,12 @@ fn merge_arg_attrs(
     }
 
     fn merge_ret_attrs(
-        doc: option<doc::retdoc>,
+        doc: doc::retdoc,
         attrs: option<str>
-    ) -> option<doc::retdoc> {
-        alt doc {
-          some(doc) {
-            some({
-                desc: attrs
-                with doc
-            })
-          }
-          none {
-            // FIXME: Warning about documenting nil?
-            none
-          }
+    ) -> doc::retdoc {
+        {
+            desc: attrs
+            with doc
         }
     }
 }
@@ -204,7 +196,7 @@ fn fold_fn_should_extract_return_attributes() {
     let doc = tystr_pass::mk_pass()(srv, doc);
     let fold = fold::default_seq_fold(srv);
     let doc = fold_fn(fold, doc.topmod.fns[0]);
-    assert option::get(doc.return).desc == some("what");
+    assert doc.return.desc == some("what");
 }
 
 #[test]

diff --git a/src/rustdoc/doc.rs b/src/rustdoc/doc.rs
index 4a3101bc71669d9ceda30212e1db458119a7e7d4..7ccd50b9649c04ba5e9f4ca6c3f36c5901351564 100644 (file)
--- a/src/rustdoc/doc.rs
+++ b/src/rustdoc/doc.rs
@@ -22,7 +22,7 @@
     brief: option<str>,
     desc: option<str>,
     args: [argdoc],
-    return: option<retdoc>,
+    return: retdoc,
     sig: option<str>
 };
 

diff --git a/src/rustdoc/extract.rs b/src/rustdoc/extract.rs
index ce6d7bdcbb43bc00f3c79956d3204b11bad64a65..9ce5bfed4fed9b489c9db68291ee35904be0200f 100644 (file)
--- a/src/rustdoc/extract.rs
+++ b/src/rustdoc/extract.rs
@@ -80,7 +80,10 @@ fn fndoc_from_fn(
         brief: none,
         desc: none,
         args: argdocs_from_args(decl.inputs),
-        return: none,
+        return: {
+            desc: none,
+            ty: none
+        },
         sig: none
     }
 }

diff --git a/src/rustdoc/gen.rs b/src/rustdoc/gen.rs
index b365de6e6f55444959064062328479295c41fe36..a355bea9934e8f07450f99300bf67fa2ce6e1fbf 100644 (file)
--- a/src/rustdoc/gen.rs
+++ b/src/rustdoc/gen.rs
@@ -222,23 +222,18 @@ fn should_write_argument_description() {
 
 fn write_return(
     ctxt: ctxt,
-    return: option<doc::retdoc>
+    doc: doc::retdoc
 ) {
-    alt return {
-      some(doc) {
-        alt doc.ty {
-          some(ty) {
-            ctxt.w.write_line(#fmt("Returns `%s`", ty));
+    alt doc.ty {
+      some(ty) {
+        ctxt.w.write_line(#fmt("Returns `%s`", ty));
+        ctxt.w.write_line("");
+        alt doc.desc {
+          some(d) {
+            ctxt.w.write_line(d);
             ctxt.w.write_line("");
-            alt doc.desc {
-              some(d) {
-                ctxt.w.write_line(d);
-                ctxt.w.write_line("");
-              }
-              none { }
-            }
           }
-          none { fail "unimplemented"; }
+          none { }
         }
       }
       none { }

diff --git a/src/rustdoc/prune_undoc_pass.rs b/src/rustdoc/prune_undoc_pass.rs
index 5bc8696b08a0392df5ada494a0656607035011b7..ba8a5456a27020d443cdf8f24083c4ddf7f7f5c7 100644 (file)
--- a/src/rustdoc/prune_undoc_pass.rs
+++ b/src/rustdoc/prune_undoc_pass.rs
@@ -32,7 +32,7 @@ fn fold_fn(
     fold.ctxt.have_docs =
         doc.brief != none
         || doc.desc != none
-        || doc.return != none;
+        || doc.return.desc != none;
     ret doc;
 }
 

diff --git a/src/rustdoc/tystr_pass.rs b/src/rustdoc/tystr_pass.rs
index d8c6c675e3a2b24c814621d43f2f2e373dc28f91..26dcaabe81665cb263b719fb6496f47c43b258af 100644 (file)
--- a/src/rustdoc/tystr_pass.rs
+++ b/src/rustdoc/tystr_pass.rs
@@ -62,23 +62,16 @@ fn should_add_fn_sig() {
 fn merge_ret_ty(
     srv: astsrv::srv,
     fn_id: doc::ast_id,
-    doc: option<doc::retdoc>
-) -> option<doc::retdoc> {
-    alt doc {
-      some(doc) {
-        fail "unimplemented";
-      }
-      none {
-        alt get_ret_ty(srv, fn_id) {
-          some(ty) {
-            some({
-                desc: none,
-                ty: some(ty)
-            })
-          }
-          none { none }
+    doc: doc::retdoc
+) -> doc::retdoc {
+    alt get_ret_ty(srv, fn_id) {
+      some(ty) {
+        {
+            ty: some(ty)
+            with doc
         }
       }
+      none { doc }
     }
 }
 
@@ -105,7 +98,7 @@ fn should_add_fn_ret_types() {
     let srv = astsrv::mk_srv_from_str(source);
     let doc = extract::from_srv(srv, "");
     let doc = run(srv, doc);
-    assert option::get(doc.topmod.fns[0].return).ty == some("int");
+    assert doc.topmod.fns[0].return.ty == some("int");
 }
 
 #[test]
@@ -114,7 +107,7 @@ fn should_not_add_nil_ret_type() {
     let srv = astsrv::mk_srv_from_str(source);
     let doc = extract::from_srv(srv, "");
     let doc = run(srv, doc);
-    assert doc.topmod.fns[0].return == none;
+    assert doc.topmod.fns[0].return.ty == none;
 }
 
 fn merge_arg_tys(