From 3c9e9d35ac0f6469e0eeffc5c497f4b3ed6f1f8a Mon Sep 17 00:00:00 2001 From: Patrick Walton Date: Wed, 15 Jan 2014 18:38:34 -0800 Subject: [PATCH] libsyntax: Remove `ident_to_str` from the parser, which was returning `@str` values --- src/libsyntax/parse/obsolete.rs | 4 ++-- src/libsyntax/parse/parser.rs | 14 ++++++-------- 2 files changed, 8 insertions(+), 10 deletions(-) diff --git a/src/libsyntax/parse/obsolete.rs b/src/libsyntax/parse/obsolete.rs index c4887d55e2a..04c73ce71d0 100644 --- a/src/libsyntax/parse/obsolete.rs +++ b/src/libsyntax/parse/obsolete.rs @@ -22,7 +22,6 @@ use parse::parser::Parser; use parse::token; -use std::str; use std::to_bytes; /// The specific types of unsupported syntax @@ -178,7 +177,8 @@ fn report(&mut self, fn is_obsolete_ident(&mut self, ident: &str) -> bool { match self.token { token::IDENT(sid, _) => { - str::eq_slice(self.id_to_str(sid), ident) + let interned_string = token::get_ident(sid.name); + interned_string.equiv(&ident) } _ => false } diff --git a/src/libsyntax/parse/parser.rs b/src/libsyntax/parse/parser.rs index 07124120b12..cdf5d440380 100644 --- a/src/libsyntax/parse/parser.rs +++ b/src/libsyntax/parse/parser.rs @@ -531,10 +531,11 @@ pub fn eat_keyword(&mut self, kw: keywords::Keyword) -> bool { // otherwise, eat it. pub fn expect_keyword(&mut self, kw: keywords::Keyword) { if !self.eat_keyword(kw) { - let id_str = self.id_to_str(kw.to_ident()).to_str(); + let id_ident = kw.to_ident(); + let id_interned_str = token::get_ident(id_ident.name); let token_str = self.this_token_to_str(); self.fatal(format!("expected `{}`, found `{}`", - id_str, + id_interned_str.get(), token_str)) } } @@ -802,10 +803,6 @@ pub fn abort_if_errors(&mut self) { self.sess.span_diagnostic.handler().abort_if_errors(); } - pub fn id_to_str(&mut self, id: Ident) -> @str { - get_ident_interner().get(id.name) - } - pub fn id_to_interned_str(&mut self, id: Ident) -> InternedString { get_ident(id.name) } @@ -3440,7 +3437,9 @@ fn parse_optional_ty_param_bounds(&mut self) -> Option> { loop { match self.token { token::LIFETIME(lifetime) => { - if "static" == self.id_to_str(lifetime) { + let lifetime_interned_string = + token::get_ident(lifetime.name); + if lifetime_interned_string.equiv(&("static")) { result.push(RegionTyParamBound); } else { self.span_err(self.span, @@ -4871,7 +4870,6 @@ fn parse_view_path(&mut self) -> @ViewPath { let first_ident = self.parse_ident(); let mut path = ~[first_ident]; - debug!("parsed view path: {}", self.id_to_str(first_ident)); match self.token { token::EQ => { // x = foo::bar -- 2.44.0