diff options
| author | bors <bors@rust-lang.org> | 2016-02-02 14:00:50 +0000 |
|---|---|---|
| committer | bors <bors@rust-lang.org> | 2016-02-02 14:00:50 +0000 |
| commit | 59b7c907a3e3dd8b263297adf6ff0515fac125e3 (patch) | |
| tree | a45c2760b88b026bf6aee2a15c259c3dd3d976c2 /src | |
| parent | ddd1bf594fdaf2f4a9ac3f03ffdb726f5121050e (diff) | |
| parent | 9a30ecdf1169dd53a1e226cc3523260ef88fd90a (diff) | |
| download | rust-59b7c907a3e3dd8b263297adf6ff0515fac125e3.tar.gz rust-59b7c907a3e3dd8b263297adf6ff0515fac125e3.zip | |
Auto merge of #31254 - tmiasko:macro-pretty-print-fix, r=sfackler
Pretty printing of macro with braces but without terminated semicolon removed more boxes from stack than it put there, resulting in panic. This fixes the issue #30731.
Diffstat (limited to 'src')
| -rw-r--r-- | src/libsyntax/print/pprust.rs | 7 | ||||
| -rw-r--r-- | src/test/pretty/issue-30731.rs | 17 |
2 files changed, 19 insertions, 5 deletions
diff --git a/src/libsyntax/print/pprust.rs b/src/libsyntax/print/pprust.rs index e80297eb797..759b16c5738 100644 --- a/src/libsyntax/print/pprust.rs +++ b/src/libsyntax/print/pprust.rs @@ -1781,11 +1781,8 @@ impl<'a> State<'a> { token::Paren => try!(self.popen()), token::Bracket => try!(word(&mut self.s, "[")), token::Brace => { - // head-ibox, will be closed by bopen() - try!(self.ibox(0)); - // Don't ask me why the regular bopen() does - // more then just opening a brace... - try!(self.bopen()) + try!(self.head("")); + try!(self.bopen()); } } try!(self.print_tts(&m.node.tts)); diff --git a/src/test/pretty/issue-30731.rs b/src/test/pretty/issue-30731.rs new file mode 100644 index 00000000000..cd72ae98433 --- /dev/null +++ b/src/test/pretty/issue-30731.rs @@ -0,0 +1,17 @@ +// Copyright 2016 The Rust Project Developers. See the COPYRIGHT +// file at the top-level directory of this distribution and at +// http://rust-lang.org/COPYRIGHT. +// +// Licensed under the Apache License, Version 2.0 <LICENSE-APACHE or +// http://www.apache.org/licenses/LICENSE-2.0> or the MIT license +// <LICENSE-MIT or http://opensource.org/licenses/MIT>, at your +// option. This file may not be copied, modified, or distributed +// except according to those terms. +// +// Test pretty printing of macro with braces but without terminating semicolon, +// this used to panic before fix. + +// pretty-compare-only +// pp-exact + +fn main() { b!{ } c } |
