auto merge of #6686 : cmr/rust/fix-6596, r=catamorphism

The error message is extremely unideal.

2 files changed, 16 insertions, 4 deletions

diff --git a/src/libsyntax/ext/tt/transcribe.rs b/src/libsyntax/ext/tt/transcribe.rs
index d806801b7d1..b13dc3f3c0f 100644
--- a/src/libsyntax/ext/tt/transcribe.rs
+++ b/src/libsyntax/ext/tt/transcribe.rs
@@ -118,10 +118,13 @@ fn lookup_cur_matched_by_matched(r: &mut TtReader,
 }
 
 fn lookup_cur_matched(r: &mut TtReader, name: ident) -> @named_match {
-    // FIXME (#3850): this looks a bit silly with an extra scope.
-    let start;
-    { start = *r.interpolations.get(&name); }
-    return lookup_cur_matched_by_matched(r, start);
+    match r.interpolations.find_copy(&name) {
+        Some(s) => lookup_cur_matched_by_matched(r, s),
+        None => {
+            r.sp_diag.span_fatal(r.cur_span, fmt!("unknown macro variable `%s`",
+                                                  *r.interner.get(name)));
+        }
+    }
 }
 enum lis {
     lis_unconstrained, lis_constraint(uint, ident), lis_contradiction(~str)
diff --git a/src/test/compile-fail/issue-6596.rs b/src/test/compile-fail/issue-6596.rs
new file mode 100644
index 00000000000..3c952dbc590
--- /dev/null
+++ b/src/test/compile-fail/issue-6596.rs
@@ -0,0 +1,9 @@
+macro_rules! e( //~ ERROR unknown macro variable `nonexistent`
+    ($inp:ident) => (
+        $nonexistent
+    );
+)
+
+fn main() {
+    e!(foo);
+}