---

yaml
---
r: 28008
b: refs/heads/try
c: 8922264
h: refs/heads/master
v: v3

Author: catamorphism

[refs]

@@ -2,7 +2,7 @@
 refs/heads/master: cd6f24f9d14ac90d167386a56e7a6ac1f0318195
 refs/heads/snap-stage1: e33de59e47c5076a89eadeb38f4934f58a3618a6
 refs/heads/snap-stage3: cd6f24f9d14ac90d167386a56e7a6ac1f0318195
-refs/heads/try: 9f591319dd46ce3ddb7ca218dafa3d072627c345
+refs/heads/try: 89222646f446e608f151c08f6623c695d7b8c673
 refs/tags/release-0.1: 1f5c5126e96c79d22cb7862f75304136e204f105
 refs/heads/ndm: f3868061cd7988080c30d6d5bf352a5a5fe2460b
 refs/heads/try2: d0c6ce338884ee21843f4b40bf6bf18d222ce5df

branches/try/src/rustdoc/attr_pass.rs

@@ -145,7 +145,7 @@ fn fold_enum(
     {
         variants: do par::map(doc.variants) |variant| {
             let desc = do astsrv::exec(srv) |ctxt| {
-                match check ctxt.ast_map.get(doc_id) {
+                match ctxt.ast_map.get(doc_id) {
                   ast_map::node_item(@{
                     node: ast::item_enum(enum_definition, _), _
                   }, _) => {
@@ -156,6 +156,8 @@ fn fold_enum(
 
                     attr_parser::parse_desc(ast_variant.node.attrs)
                   }
+                  _ => fail #fmt("Enum variant %s has id that's not bound \
+                         to an enum item", variant.name)
                 }
             };
 

branches/try/src/rustdoc/tystr_pass.rs

@@ -47,7 +47,7 @@ fn fold_fn(
 
 fn get_fn_sig(srv: astsrv::srv, fn_id: doc::ast_id) -> option<~str> {
     do astsrv::exec(srv) |ctxt| {
-        match check ctxt.ast_map.get(fn_id) {
+        match ctxt.ast_map.get(fn_id) {
           ast_map::node_item(@{
             ident: ident,
             node: ast::item_fn(decl, tys, _), _
@@ -58,6 +58,7 @@ fn get_fn_sig(srv: astsrv::srv, fn_id: doc::ast_id) -> option<~str> {
           }, _, _) => {
             some(pprust::fun_to_str(decl, ident, tys, extract::interner()))
           }
+          _ => fail ~"get_fn_sig: fn_id not bound to a fn item"
         }
     }
 }
@@ -82,12 +83,13 @@ fn fold_const(
 
     {
         sig: some(do astsrv::exec(srv) |ctxt| {
-            match check ctxt.ast_map.get(doc.id()) {
+            match ctxt.ast_map.get(doc.id()) {
               ast_map::node_item(@{
                 node: ast::item_const(ty, _), _
               }, _) => {
                 pprust::ty_to_str(ty, extract::interner())
               }
+              _ => fail ~"fold_const: id not bound to a const item"
             }
         })
         with doc
@@ -110,7 +112,7 @@ fn fold_enum(
     {
         variants: do par::map(doc.variants) |variant| {
             let sig = do astsrv::exec(srv) |ctxt| {
-                match check ctxt.ast_map.get(doc_id) {
+                match ctxt.ast_map.get(doc_id) {
                   ast_map::node_item(@{
                     node: ast::item_enum(enum_definition, _), _
                   }, _) => {
@@ -121,6 +123,7 @@ fn fold_enum(
 
                     pprust::variant_to_str(ast_variant, extract::interner())
                   }
+                  _ => fail ~"enum variant not bound to an enum item"
                 }
             };
 
@@ -168,11 +171,11 @@ fn get_method_sig(
     method_name: ~str
 ) -> option<~str> {
     do astsrv::exec(srv) |ctxt| {
-        match check ctxt.ast_map.get(item_id) {
+        match ctxt.ast_map.get(item_id) {
           ast_map::node_item(@{
             node: ast::item_trait(_, _, methods), _
           }, _) => {
-            match check vec::find(methods, |method| {
+            match vec::find(methods, |method| {
                 match method {
                   ast::required(ty_m) => to_str(ty_m.ident) == method_name,
                   ast::provided(m) => to_str(m.ident) == method_name,
@@ -198,12 +201,13 @@ fn get_method_sig(
                     }
                   }
                 }
+                _ => fail ~"method not found"
             }
           }
           ast_map::node_item(@{
             node: ast::item_impl(_, _, _, methods), _
           }, _) => {
-            match check vec::find(methods, |method| {
+            match vec::find(methods, |method| {
                 to_str(method.ident) == method_name
             }) {
                 some(method) => {
@@ -214,8 +218,10 @@ fn get_method_sig(
                         extract::interner()
                     ))
                 }
+                none => fail ~"method not found"
             }
           }
+          _ => fail ~"get_method_sig: item ID not bound to trait or impl"
         }
     }
 }