---

yaml
---
r: 13915
b: refs/heads/try
c: 2c495a9
h: refs/heads/master
i:
  13913: fe6a993
  13911: 3e0cde9
v: v3

Author: brson

[refs]

@@ -2,5 +2,5 @@
 refs/heads/master: 61b1875c16de39c166b0f4d54bba19f9c6777d1a
 refs/heads/snap-stage1: e33de59e47c5076a89eadeb38f4934f58a3618a6
 refs/heads/snap-stage3: 4a81779abd786ff22d71434c6d9a5917ea4cdfff
-refs/heads/try: b8a683415c995bdb3d2e8bdc0d6a2bec7fe6cf1d
+refs/heads/try: 2c495a9bf8c1cf6cf76d41b5aa0dad4998fb6c57
 refs/tags/release-0.1: 1f5c5126e96c79d22cb7862f75304136e204f105

branches/try/src/rustdoc/attr_pass.rs

@@ -25,7 +25,8 @@ fn run(
         fold_fn: fold_fn,
         fold_const: fold_const,
         fold_enum: fold_enum,
-        fold_res: fold_res
+        fold_res: fold_res,
+        fold_iface: fold_iface
         with *fold::default_seq_fold(srv)
     });
     fold.fold_crate(fold, doc)
@@ -142,35 +143,36 @@ fn fold_fn(
             with doc
         };
     }
+}
 
-    fn merge_arg_attrs(
-        docs: [doc::argdoc],
-        attrs: [attr_parser::arg_attrs]
-    ) -> [doc::argdoc] {
-        vec::map(docs) {|doc|
-            alt vec::find(attrs) {|attr|
-                attr.name == doc.name
-            } {
-                some(attr) {
-                    {
-                        desc: some(attr.desc)
-                        with doc
-                    }
+fn merge_arg_attrs(
+    docs: [doc::argdoc],
+    attrs: [attr_parser::arg_attrs]
+) -> [doc::argdoc] {
+    vec::map(docs) {|doc|
+        alt vec::find(attrs) {|attr|
+            attr.name == doc.name
+        } {
+            some(attr) {
+                {
+                    desc: some(attr.desc)
+                    with doc
                 }
-                none { doc }
             }
+            none { doc }
         }
-        // FIXME: Warning when documenting a non-existent arg
     }
+    // FIXME: Warning when documenting a non-existent arg
+}
 
-    fn merge_ret_attrs(
-        doc: doc::retdoc,
-        attrs: option<str>
-    ) -> doc::retdoc {
-        {
-            desc: attrs
-            with doc
-        }
+
+fn merge_ret_attrs(
+    doc: doc::retdoc,
+    attrs: option<str>
+) -> doc::retdoc {
+    {
+        desc: attrs
+        with doc
     }
 }
 
@@ -359,4 +361,84 @@ fn fold_res_should_extract_arg_docs() {
     let doc = fold_res(fold, doc.topmod.resources()[0]);
     assert doc.args[0].name == "a";
     assert doc.args[0].desc == some("b");
+}
+
+
+fn fold_iface(
+    fold: fold::fold<astsrv::srv>,
+    doc: doc::ifacedoc
+) -> doc::ifacedoc {
+    let srv = fold.ctxt;
+    let doc = fold::default_seq_fold_iface(fold, doc);
+    let attrs = parse_item_attrs(srv, doc.id, attr_parser::parse_iface);
+
+    {
+        brief: attrs.brief,
+        desc: attrs.desc,
+        methods: merge_method_attrs(srv, doc.id, doc.methods)
+        with doc
+    }
+}
+
+fn merge_method_attrs(
+    srv: astsrv::srv,
+    item_id: doc::ast_id,
+    docs: [doc::methoddoc]
+) -> [doc::methoddoc] {
+    // Create an assoc list from method name to attributes
+    let attrs = astsrv::exec(srv) {|ctxt|
+        alt ctxt.ast_map.get(item_id) {
+          ast_map::node_item(@{
+            node: ast::item_iface(_, methods), _
+          }) {
+            vec::map(methods) {|method|
+                (method.ident, attr_parser::parse_method(method.attrs))
+            }
+          }
+        }
+    };
+
+    vec::map2(docs, attrs) {|doc, attrs|
+        assert doc.name == tuple::first(attrs);
+        let attrs = tuple::second(attrs);
+
+        {
+            brief: attrs.brief,
+            desc: attrs.desc,
+            args: merge_arg_attrs(doc.args, attrs.args),
+            return: merge_ret_attrs(doc.return, attrs.return),
+            failure: attrs.failure
+            with doc
+        }
+    }
+}
+
+#[test]
+fn should_extract_iface_docs() {
+    let source = "#[doc = \"whatever\"] iface i { fn a(); }";
+    let srv = astsrv::mk_srv_from_str(source);
+    let doc = extract::from_srv(srv, "");
+    let doc = run(srv, doc);
+    assert doc.topmod.ifaces()[0].desc == some("whatever");
+}
+
+#[test]
+fn should_extract_iface_method_docs() {
+    let source = "iface i {\
+                  #[doc(\
+                  brief = \"brief\",\
+                  desc = \"desc\",\
+                  args(a = \"a\"),\
+                  return = \"return\",\
+                  failure = \"failure\")]\
+                  fn f(a: bool) -> bool;\
+                  }";
+    let srv = astsrv::mk_srv_from_str(source);
+    let doc = extract::from_srv(srv, "");
+    let doc = run(srv, doc);
+    assert doc.topmod.ifaces()[0].methods[0].brief == some("brief");
+    assert doc.topmod.ifaces()[0].methods[0].desc == some("desc");
+    assert doc.topmod.ifaces()[0].methods[0].args[0].desc == some("a");
+    assert doc.topmod.ifaces()[0].methods[0].return.desc == some("return");
+    assert doc.topmod.ifaces()[0].methods[0].failure == some("failure");
 }