From 7fbcb51589a9d10a2b3ec227611b31b7384b5370 Mon Sep 17 00:00:00 2001 From: Manish Goregaokar Date: Tue, 24 Nov 2015 07:23:53 +0530 Subject: [PATCH] Fix unreachable code in libsyntax --- src/libsyntax/ext/base.rs | 4 ++-- src/libsyntax/ext/tt/macro_rules.rs | 4 ++-- src/test/compile-fail/unreachable-in-call.rs | 5 ----- 3 files changed, 4 insertions(+), 9 deletions(-) diff --git a/src/libsyntax/ext/base.rs b/src/libsyntax/ext/base.rs index 0dba15760cd..08888b3073d 100644 --- a/src/libsyntax/ext/base.rs +++ b/src/libsyntax/ext/base.rs @@ -678,9 +678,9 @@ impl<'a> ExtCtxt<'a> { pub fn bt_push(&mut self, ei: ExpnInfo) { self.recursion_count += 1; if self.recursion_count > self.ecfg.recursion_limit { - panic!(self.span_fatal(ei.call_site, + self.span_fatal(ei.call_site, &format!("recursion limit reached while expanding the macro `{}`", - ei.callee.name()))); + ei.callee.name())); } let mut call_site = ei.call_site; diff --git a/src/libsyntax/ext/tt/macro_rules.rs b/src/libsyntax/ext/tt/macro_rules.rs index 43647ea4a2e..23d6b794a72 100644 --- a/src/libsyntax/ext/tt/macro_rules.rs +++ b/src/libsyntax/ext/tt/macro_rules.rs @@ -209,12 +209,12 @@ fn generic_extension<'cx>(cx: &'cx ExtCtxt, best_fail_msg = (*msg).clone(); }, Error(err_sp, ref msg) => { - panic!(cx.span_fatal(err_sp.substitute_dummy(sp), &msg[..])) + cx.span_fatal(err_sp.substitute_dummy(sp), &msg[..]) } } } - panic!(cx.span_fatal(best_fail_spot.substitute_dummy(sp), &best_fail_msg[..])); + cx.span_fatal(best_fail_spot.substitute_dummy(sp), &best_fail_msg[..]); } // Note that macro-by-example's input is also matched against a token tree: diff --git a/src/test/compile-fail/unreachable-in-call.rs b/src/test/compile-fail/unreachable-in-call.rs index 61c89e3a4b4..5a3257d54db 100644 --- a/src/test/compile-fail/unreachable-in-call.rs +++ b/src/test/compile-fail/unreachable-in-call.rs @@ -8,11 +8,6 @@ // option. This file may not be copied, modified, or distributed // except according to those terms. -// Test that the unboxed closure sugar can be used with an arbitrary -// struct type and that it is equivalent to the same syntax using -// angle brackets. This test covers only simple types and in -// particular doesn't test bound regions. - #![allow(dead_code)] #![deny(unreachable_code)]