Skip to content

Commit

Permalink
Don't ICE on macros with -Z show-span
Browse files Browse the repository at this point in the history
Closes #17115
  • Loading branch information
sfackler committed Sep 10, 2014
1 parent 641b198 commit b865c1f
Showing 1 changed file with 4 additions and 0 deletions.
4 changes: 4 additions & 0 deletions src/librustc/front/show_span.rs
Expand Up @@ -28,6 +28,10 @@ impl<'a> Visitor<()> for ShowSpanVisitor<'a> {
self.sess.span_note(e.span, "expression");
visit::walk_expr(self, e, ());
}

fn visit_mac(&mut self, macro: &ast::Mac, e: ()) {
visit::walk_mac(self, macro, e);
}
}

pub fn run(sess: &Session, krate: &ast::Crate) {
Expand Down

9 comments on commit b865c1f

@bors
Copy link
Contributor

@bors bors commented on b865c1f Sep 10, 2014

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

saw approval from alexcrichton
at sfackler@b865c1f

@bors
Copy link
Contributor

@bors bors commented on b865c1f Sep 10, 2014

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

merging sfackler/rust/issue-17115 = b865c1f into auto

@bors
Copy link
Contributor

@bors bors commented on b865c1f Sep 10, 2014

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

sfackler/rust/issue-17115 = b865c1f merged ok, testing candidate = d0a75d13

@bors
Copy link
Contributor

@bors bors commented on b865c1f Sep 11, 2014

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

saw approval from alexcrichton
at sfackler@b865c1f

@bors
Copy link
Contributor

@bors bors commented on b865c1f Sep 11, 2014

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

merging sfackler/rust/issue-17115 = b865c1f into auto

@bors
Copy link
Contributor

@bors bors commented on b865c1f Sep 11, 2014

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

sfackler/rust/issue-17115 = b865c1f merged ok, testing candidate = 7ea660e

@bors
Copy link
Contributor

@bors bors commented on b865c1f Sep 11, 2014

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@bors
Copy link
Contributor

@bors bors commented on b865c1f Sep 11, 2014

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

fast-forwarding master to auto = 7ea660e

Please sign in to comment.