diff options
| author | Alex Crichton <alex@alexcrichton.com> | 2013-10-21 13:08:31 -0700 |
|---|---|---|
| committer | Alex Crichton <alex@alexcrichton.com> | 2013-10-22 08:09:56 -0700 |
| commit | daf5f5a4d10513ff42e79fa7ef8819b170f3a13d (patch) | |
| tree | 7a07a79c43e02debcc6bbb33d90a5e41b70119e6 /src/libsyntax/ext | |
| parent | 15a6bdebab4e7b811b9a902e3f8ed225c59af06e (diff) | |
| download | rust-daf5f5a4d10513ff42e79fa7ef8819b170f3a13d.tar.gz rust-daf5f5a4d10513ff42e79fa7ef8819b170f3a13d.zip | |
Drop the '2' suffix from logging macros
Who doesn't like a massive renaming?
Diffstat (limited to 'src/libsyntax/ext')
| -rw-r--r-- | src/libsyntax/ext/base.rs | 6 | ||||
| -rw-r--r-- | src/libsyntax/ext/expand.rs | 42 | ||||
| -rw-r--r-- | src/libsyntax/ext/quote.rs | 10 | ||||
| -rw-r--r-- | src/libsyntax/ext/tt/macro_parser.rs | 6 |
4 files changed, 32 insertions, 32 deletions
diff --git a/src/libsyntax/ext/base.rs b/src/libsyntax/ext/base.rs index 78c4d6b6f4b..1f9fe28a46d 100644 --- a/src/libsyntax/ext/base.rs +++ b/src/libsyntax/ext/base.rs @@ -540,11 +540,11 @@ impl <K: Eq + Hash + IterBytes + 'static, V: 'static> MapChain<K,V>{ // names? I think not. // delaying implementing this.... pub fn each_key (&self, _f: &fn (&K)->bool) { - fail2!("unimplemented 2013-02-15T10:01"); + fail!("unimplemented 2013-02-15T10:01"); } pub fn each_value (&self, _f: &fn (&V) -> bool) { - fail2!("unimplemented 2013-02-15T10:02"); + fail!("unimplemented 2013-02-15T10:02"); } // Returns a copy of the value that the name maps to. @@ -587,7 +587,7 @@ impl <K: Eq + Hash + IterBytes + 'static, V: 'static> MapChain<K,V>{ if satisfies_pred(map,&n,pred) { map.insert(key,ext); } else { - fail2!("expected map chain containing satisfying frame") + fail!("expected map chain containing satisfying frame") } }, ConsMapChain (~ref mut map, rest) => { diff --git a/src/libsyntax/ext/expand.rs b/src/libsyntax/ext/expand.rs index 620594a0a17..e1cc63fd0b8 100644 --- a/src/libsyntax/ext/expand.rs +++ b/src/libsyntax/ext/expand.rs @@ -429,7 +429,7 @@ fn insert_macro(exts: SyntaxEnv, name: ast::Name, transformer: @Transformer) { match t { &@BlockInfo(BlockInfo {macros_escape:false,_}) => true, &@BlockInfo(BlockInfo {_}) => false, - _ => fail2!("special identifier {:?} was bound to a non-BlockInfo", + _ => fail!("special identifier {:?} was bound to a non-BlockInfo", special_block_name) } }; @@ -741,7 +741,7 @@ pub fn expand_block_elts(exts: SyntaxEnv, b: &Block, fld: &MacroExpander) fn mustbesome<T>(val : Option<T>) -> T { match val { Some(v) => v, - None => fail2!("rename_fold returned None") + None => fail!("rename_fold returned None") } } @@ -749,7 +749,7 @@ fn mustbesome<T>(val : Option<T>) -> T { fn get_block_info(exts : SyntaxEnv) -> BlockInfo { match exts.find_in_topmost_frame(&intern(special_block_name)) { Some(@BlockInfo(bi)) => bi, - _ => fail2!("special identifier {:?} was bound to a non-BlockInfo", + _ => fail!("special identifier {:?} was bound to a non-BlockInfo", @" block") } } @@ -782,7 +782,7 @@ pub fn renames_to_fold(renames: @mut ~[(ast::Ident,ast::Name)]) -> @ast_fold { fn apply_pending_renames(folder : @ast_fold, stmt : ast::Stmt) -> @ast::Stmt { match folder.fold_stmt(&stmt) { Some(s) => s, - None => fail2!("renaming of stmt produced None") + None => fail!("renaming of stmt produced None") } } @@ -840,11 +840,11 @@ pub fn std_macros() -> @str { // NOTE (acrichto): remove these after the next snapshot macro_rules! log2( ($($arg:tt)*) => (log!($($arg)*)) ) - macro_rules! error2( ($($arg:tt)*) => (error!($($arg)*)) ) - macro_rules! warn2 ( ($($arg:tt)*) => (warn!($($arg)*)) ) - macro_rules! info2 ( ($($arg:tt)*) => (info!($($arg)*)) ) - macro_rules! debug2( ($($arg:tt)*) => (debug!($($arg)*)) ) - macro_rules! fail2( ($($arg:tt)*) => (fail!($($arg)*)) ) + macro_rules! error( ($($arg:tt)*) => (error!($($arg)*)) ) + macro_rules! warn ( ($($arg:tt)*) => (warn!($($arg)*)) ) + macro_rules! info ( ($($arg:tt)*) => (info!($($arg)*)) ) + macro_rules! debug( ($($arg:tt)*) => (debug!($($arg)*)) ) + macro_rules! fail( ($($arg:tt)*) => (fail!($($arg)*)) ) macro_rules! assert( ($cond:expr) => { @@ -873,7 +873,7 @@ pub fn std_macros() -> @str { // check both directions of equality.... if !((*given_val == *expected_val) && (*expected_val == *given_val)) { - fail2!(\"assertion failed: `(left == right) && (right == \ + fail!(\"assertion failed: `(left == right) && (right == \ left)` (left: `{:?}`, right: `{:?}`)\", *given_val, *expected_val); } @@ -893,7 +893,7 @@ pub fn std_macros() -> @str { given_val.approx_eq(&expected_val) && expected_val.approx_eq(&given_val) ) { - fail2!(\"left: {:?} does not approximately equal right: {:?}\", + fail!(\"left: {:?} does not approximately equal right: {:?}\", given_val, expected_val); } } @@ -910,7 +910,7 @@ pub fn std_macros() -> @str { given_val.approx_eq_eps(&expected_val, &epsilon_val) && expected_val.approx_eq_eps(&given_val, &epsilon_val) ) { - fail2!(\"left: {:?} does not approximately equal right: \ + fail!(\"left: {:?} does not approximately equal right: \ {:?} with epsilon: {:?}\", given_val, expected_val, epsilon_val); } @@ -945,7 +945,7 @@ pub fn std_macros() -> @str { */ macro_rules! unreachable (() => ( - fail2!(\"internal error: entered unreachable code\"); + fail!(\"internal error: entered unreachable code\"); )) macro_rules! condition ( @@ -1123,7 +1123,7 @@ pub fn inject_std_macros(parse_sess: @mut parse::ParseSess, ~[], parse_sess) { Some(item) => item, - None => fail2!("expected core macros to parse correctly") + None => fail!("expected core macros to parse correctly") }; let injector = @Injector { @@ -1381,16 +1381,16 @@ mod test { use util::parser_testing::{string_to_pat, string_to_tts, strs_to_idents}; use visit; - // make sure that fail2! is present + // make sure that fail! is present #[test] fn fail_exists_test () { - let src = @"fn main() { fail2!(\"something appropriately gloomy\");}"; + let src = @"fn main() { fail!(\"something appropriately gloomy\");}"; let sess = parse::new_parse_sess(None); let crate_ast = parse::parse_crate_from_source_str( @"<test>", src, ~[],sess); let crate_ast = inject_std_macros(sess, ~[], crate_ast); - // don't bother with striping, doesn't affect fail2!. + // don't bother with striping, doesn't affect fail!. expand_crate(sess,~[],crate_ast); } @@ -1448,7 +1448,7 @@ mod test { cfg,~[],sess); match item_ast { Some(_) => (), // success - None => fail2!("expected this to parse") + None => fail!("expected this to parse") } } @@ -1487,7 +1487,7 @@ mod test { let marked_once_ctxt = match marked_once[0] { ast::tt_tok(_,token::IDENT(id,_)) => id.ctxt, - _ => fail2!(format!("unexpected shape for marked tts: {:?}",marked_once[0])) + _ => fail!(format!("unexpected shape for marked tts: {:?}",marked_once[0])) }; assert_eq!(mtwt_marksof(marked_once_ctxt,invalid_name),~[fm]); let remarked = mtwt_cancel_outer_mark(marked_once,marked_once_ctxt); @@ -1495,7 +1495,7 @@ mod test { match remarked[0] { ast::tt_tok(_,token::IDENT(id,_)) => assert_eq!(mtwt_marksof(id.ctxt,invalid_name),~[]), - _ => fail2!(format!("unexpected shape for marked tts: {:?}",remarked[0])) + _ => fail!(format!("unexpected shape for marked tts: {:?}",remarked[0])) } } @@ -1700,7 +1700,7 @@ foo_module!() bindings.iter().filter(|b|{@"xx" == (ident_to_str(*b))}).collect(); let cxbind = match cxbinds { [b] => b, - _ => fail2!("expected just one binding for ext_cx") + _ => fail!("expected just one binding for ext_cx") }; let resolved_binding = mtwt_resolve(*cxbind); // find all the xx varrefs: diff --git a/src/libsyntax/ext/quote.rs b/src/libsyntax/ext/quote.rs index 4bef9601855..d10ec422987 100644 --- a/src/libsyntax/ext/quote.rs +++ b/src/libsyntax/ext/quote.rs @@ -255,8 +255,8 @@ pub mod rt { match res { Some(ast) => ast, None => { - error2!("Parse error with ```\n{}\n```", s); - fail2!() + error!("Parse error with ```\n{}\n```", s); + fail!() } } } @@ -490,7 +490,7 @@ fn mk_token(cx: @ExtCtxt, sp: Span, tok: &token::Token) -> @ast::Expr { ~[mk_ident(cx, sp, ident)]); } - INTERPOLATED(_) => fail2!("quote! with interpolated token"), + INTERPOLATED(_) => fail!("quote! with interpolated token"), _ => () } @@ -528,7 +528,7 @@ fn mk_token(cx: @ExtCtxt, sp: Span, tok: &token::Token) -> @ast::Expr { DOLLAR => "DOLLAR", UNDERSCORE => "UNDERSCORE", EOF => "EOF", - _ => fail2!() + _ => fail!() }; cx.expr_ident(sp, id_ext(name)) } @@ -553,7 +553,7 @@ fn mk_tt(cx: @ExtCtxt, sp: Span, tt: &ast::token_tree) } ast::tt_delim(ref tts) => mk_tts(cx, sp, **tts), - ast::tt_seq(*) => fail2!("tt_seq in quote!"), + ast::tt_seq(*) => fail!("tt_seq in quote!"), ast::tt_nonterminal(sp, ident) => { diff --git a/src/libsyntax/ext/tt/macro_parser.rs b/src/libsyntax/ext/tt/macro_parser.rs index 7db64feb809..50688afc56a 100644 --- a/src/libsyntax/ext/tt/macro_parser.rs +++ b/src/libsyntax/ext/tt/macro_parser.rs @@ -122,7 +122,7 @@ pub struct MatcherPos { pub fn copy_up(mpu: &matcher_pos_up) -> ~MatcherPos { match *mpu { matcher_pos_up(Some(ref mp)) => (*mp).clone(), - _ => fail2!() + _ => fail!() } } @@ -387,7 +387,7 @@ pub fn parse( format!("{} ('{}')", ident_to_str(name), ident_to_str(bind)) } - _ => fail2!() + _ => fail!() } }).connect(" or "); return error(sp, format!( "Local ambiguity: multiple parsing options: \ @@ -412,7 +412,7 @@ pub fn parse( parse_nt(&rust_parser, ident_to_str(name)))); ei.idx += 1u; } - _ => fail2!() + _ => fail!() } cur_eis.push(ei); |
