---

yaml
---
r: 31130
b: refs/heads/dist-snap
c: 46fba10
h: refs/heads/master
v: v3

Author: brson

[refs]

@@ -7,6 +7,6 @@ refs/tags/release-0.1: 1f5c5126e96c79d22cb7862f75304136e204f105
 refs/heads/ndm: f3868061cd7988080c30d6d5bf352a5a5fe2460b
 refs/heads/try2: d0c6ce338884ee21843f4b40bf6bf18d222ce5df
 refs/heads/incoming: d9317a174e434d4c99fc1a37fd7dc0d2f5328d37
-refs/heads/dist-snap: 200a2ded3245eb0a1ca7b265ce83adba16d75b97
+refs/heads/dist-snap: 46fba10fe82ef9a584118b8d2298cdfc49320c85
 refs/tags/release-0.2: c870d2dffb391e14efb05aa27898f1f6333a9596
 refs/tags/release-0.3: b5f0d0f648d9a6153664837026ba1be43d3e2503

branches/dist-snap/src/rustc/middle/trans/reachable.rs

@@ -27,7 +27,7 @@ fn find_reachable(crate_mod: _mod, exp_map: resolve::exp_map,
     let rmap = std::map::int_hash();
     let cx = {exp_map: exp_map, tcx: tcx, method_map: method_map, rmap: rmap};
     traverse_public_mod(cx, crate_mod);
-    traverse_all_resources(cx, crate_mod);
+    traverse_all_resources_and_impls(cx, crate_mod);
     rmap
 }
 
@@ -81,18 +81,6 @@ fn traverse_public_mod(cx: ctx, m: _mod) {
     if !traverse_exports(cx, m.view_items) {
         // No exports, so every local item is exported
         for vec::each(m.items) |item| { traverse_public_item(cx, item); }
-    } else {
-        // Make impls always reachable.
-        for vec::each(m.items) |item| {
-            alt item.node {
-                item_impl(*) {
-                    traverse_public_item(cx, item);
-                }
-                _ {
-                    // Nothing to do.
-                }
-            }
-        }
     }
 }
 
@@ -212,7 +200,7 @@ fn traverse_inline_body(cx: ctx, body: blk) {
     }));
 }
 
-fn traverse_all_resources(cx: ctx, crate_mod: _mod) {
+fn traverse_all_resources_and_impls(cx: ctx, crate_mod: _mod) {
     visit::visit_mod(crate_mod, ast_util::dummy_sp(), 0, cx, visit::mk_vt(@{
         visit_expr: |_e, _cx, _v| { },
         visit_item: |i, cx, v| {
@@ -221,9 +209,13 @@ fn traverse_all_resources(cx: ctx, crate_mod: _mod) {
               item_class(_, _, _, _, some(_)) {
                 traverse_public_item(cx, i);
               }
+              item_impl(*) {
+                traverse_public_item(cx, i);
+              }
               _ {}
             }
         }
         with *visit::default_visitor()
     }));
 }
+

branches/dist-snap/src/test/auxiliary/crate-method-reexport-grrrrrrr2.rs

@@ -0,0 +1,28 @@
+#[link(name = "crate_method_reexport_grrrrrrr2")];
+
+export rust;
+
+import name_pool::methods;
+
+mod name_pool {
+
+    type name_pool = ();
+
+    impl methods for name_pool {
+        fn add(s: str) {
+        }
+    }
+}
+
+mod rust {
+
+    export rt;
+    export methods;
+
+    type rt = @();
+
+    impl methods for rt {
+        fn cx() {
+        }
+    }
+}

branches/dist-snap/src/test/run-pass/crate-method-reexport-grrrrrrr.rs

@@ -0,0 +1,15 @@
+// This is a regression test that the metadata for the
+// name_pool::methods impl in the other crate is reachable from this
+// crate.
+
+// aux-build:crate-method-reexport-grrrrrrr2.rs
+
+use crate_method_reexport_grrrrrrr2;
+
+fn main() {
+    import crate_method_reexport_grrrrrrr2::rust::methods;
+    let x = @();
+    x.cx();
+    let y = ();
+    y.add("hi");
+}