Skip to content

Commit 5ea959d

Browse files
committed
Convert symbols to strings for pretty printing
1 parent 438f6b0 commit 5ea959d

File tree

3 files changed

+14
-17
lines changed

3 files changed

+14
-17
lines changed

src/librustc/hir/print.rs

+3-3
Original file line numberDiff line numberDiff line change
@@ -622,7 +622,7 @@ impl<'a> State<'a> {
622622
}
623623
hir::ItemKind::GlobalAsm(ref ga) => {
624624
self.head(visibility_qualified(&item.vis, "global asm"))?;
625-
self.s.word(ga.asm.as_str().get())?;
625+
self.s.word(ga.asm.as_str().to_string())?;
626626
self.end()?
627627
}
628628
hir::ItemKind::Ty(ref ty, ref generics) => {
@@ -1591,7 +1591,7 @@ impl<'a> State<'a> {
15911591
if ident.is_raw_guess() {
15921592
self.s.word(format!("r#{}", ident.name))?;
15931593
} else {
1594-
self.s.word(ident.as_str().get())?;
1594+
self.s.word(ident.as_str().to_string())?;
15951595
}
15961596
self.ann.post(self, AnnNode::Name(&ident.name))
15971597
}
@@ -1998,7 +1998,7 @@ impl<'a> State<'a> {
19981998
self.commasep(Inconsistent, &decl.inputs, |s, ty| {
19991999
s.ibox(indent_unit)?;
20002000
if let Some(arg_name) = arg_names.get(i) {
2001-
s.s.word(arg_name.as_str().get())?;
2001+
s.s.word(arg_name.as_str().to_string())?;
20022002
s.s.word(":")?;
20032003
s.s.space()?;
20042004
} else if let Some(body_id) = body_id {

src/libsyntax/print/pp.rs

+4-7
Original file line numberDiff line numberDiff line change
@@ -369,7 +369,7 @@ impl<'a> Printer<'a> {
369369
Ok(())
370370
}
371371

372-
fn pretty_print_string<'s>(&mut self, s: Cow<'s, str>, len: isize) -> io::Result<()> {
372+
fn pretty_print_string(&mut self, s: Cow<'static, str>, len: isize) -> io::Result<()> {
373373
if self.scan_stack.is_empty() {
374374
debug!("pp String('{}')/print Vec<{},{}>",
375375
s, self.left, self.right);
@@ -378,10 +378,7 @@ impl<'a> Printer<'a> {
378378
debug!("pp String('{}')/buffer Vec<{},{}>",
379379
s, self.left, self.right);
380380
self.advance_right();
381-
self.buf[self.right] = BufEntry {
382-
token: Token::String(s.into_owned().into(), len),
383-
size: len
384-
};
381+
self.buf[self.right] = BufEntry { token: Token::String(s, len), size: len };
385382
self.right_total += len;
386383
self.check_stream()
387384
}
@@ -579,7 +576,7 @@ impl<'a> Printer<'a> {
579576
}
580577
}
581578

582-
pub fn print_string(&mut self, s: Cow<'_, str>, len: isize) -> io::Result<()> {
579+
pub fn print_string(&mut self, s: Cow<'static, str>, len: isize) -> io::Result<()> {
583580
debug!("print String({})", s);
584581
// assert!(len <= space);
585582
self.space -= len;
@@ -644,7 +641,7 @@ impl<'a> Printer<'a> {
644641
self.pretty_print_eof()
645642
}
646643

647-
pub fn word<'s, S: Into<Cow<'s, str>>>(&mut self, wrd: S) -> io::Result<()> {
644+
pub fn word<S: Into<Cow<'static, str>>>(&mut self, wrd: S) -> io::Result<()> {
648645
let s = wrd.into();
649646
let len = s.len() as isize;
650647
self.pretty_print_string(s, len)

src/libsyntax/print/pprust.rs

+7-7
Original file line numberDiff line numberDiff line change
@@ -645,7 +645,7 @@ pub trait PrintState<'a> {
645645
ast::LitKind::Float(ref f, t) => {
646646
self.writer().word(format!("{}{}", &f, t.ty_to_string()))
647647
}
648-
ast::LitKind::FloatUnsuffixed(ref f) => self.writer().word(f.as_str().get()),
648+
ast::LitKind::FloatUnsuffixed(ref f) => self.writer().word(f.as_str().to_string()),
649649
ast::LitKind::Bool(val) => {
650650
if val { self.writer().word("true") } else { self.writer().word("false") }
651651
}
@@ -731,7 +731,7 @@ pub trait PrintState<'a> {
731731
if segment.ident.name == keywords::DollarCrate.name() {
732732
self.print_dollar_crate(segment.ident)?;
733733
} else {
734-
self.writer().word(segment.ident.as_str().get())?;
734+
self.writer().word(segment.ident.as_str().to_string())?;
735735
}
736736
}
737737
}
@@ -749,7 +749,7 @@ pub trait PrintState<'a> {
749749
}
750750
self.maybe_print_comment(attr.span.lo())?;
751751
if attr.is_sugared_doc {
752-
self.writer().word(attr.value_str().unwrap().as_str().get())?;
752+
self.writer().word(attr.value_str().unwrap().as_str().to_string())?;
753753
self.writer().hardbreak()
754754
} else {
755755
match attr.style {
@@ -858,7 +858,7 @@ pub trait PrintState<'a> {
858858
if !ast::Ident::with_empty_ctxt(name).is_path_segment_keyword() {
859859
self.writer().word("::")?;
860860
}
861-
self.writer().word(name.as_str().get())
861+
self.writer().word(name.as_str().to_string())
862862
}
863863
}
864864

@@ -1300,7 +1300,7 @@ impl<'a> State<'a> {
13001300
}
13011301
ast::ItemKind::GlobalAsm(ref ga) => {
13021302
self.head(visibility_qualified(&item.vis, "global_asm!"))?;
1303-
self.s.word(ga.asm.as_str().get())?;
1303+
self.s.word(ga.asm.as_str().to_string())?;
13041304
self.end()?;
13051305
}
13061306
ast::ItemKind::Ty(ref ty, ref generics) => {
@@ -2437,7 +2437,7 @@ impl<'a> State<'a> {
24372437
if ident.is_raw_guess() {
24382438
self.s.word(format!("r#{}", ident))?;
24392439
} else {
2440-
self.s.word(ident.as_str().get())?;
2440+
self.s.word(ident.as_str().to_string())?;
24412441
}
24422442
self.ann.post(self, AnnNode::Ident(&ident))
24432443
}
@@ -2447,7 +2447,7 @@ impl<'a> State<'a> {
24472447
}
24482448

24492449
pub fn print_name(&mut self, name: ast::Name) -> io::Result<()> {
2450-
self.s.word(name.as_str().get())?;
2450+
self.s.word(name.as_str().to_string())?;
24512451
self.ann.post(self, AnnNode::Name(&name))
24522452
}
24532453

0 commit comments

Comments
 (0)