Fix the most egregious instances of "local ambiguity: multiple parsing options..." error in macros, which often occurs when trying to match parts of Rust syntax.
For example, this matcher: `fn $name:ident( $($param:ident : $pty:ty),* )` would fail when parsing `fn foo()`, because macro parser wouldn't realize that an ident cannot start with `)`. This resolves #5902, and at least partially mitigates #9364 and #3232.
This commit is contained in:
parent
e62ef37cfa
commit
fc60797f7f
@ -350,7 +350,16 @@ pub fn parse(sess: &ParseSess,
|
||||
sp_lo: sp.lo
|
||||
});
|
||||
}
|
||||
MatchNonterminal(_,_,_) => { bb_eis.push(ei) }
|
||||
MatchNonterminal(_,_,_) => {
|
||||
// Built-in nonterminals never start with these tokens,
|
||||
// so we can eliminate them from consideration.
|
||||
match tok {
|
||||
token::RPAREN |
|
||||
token::RBRACE |
|
||||
token::RBRACKET => {},
|
||||
_ => bb_eis.push(ei)
|
||||
}
|
||||
}
|
||||
MatchTok(ref t) => {
|
||||
let mut ei_t = ei.clone();
|
||||
if token_name_eq(t,&tok) {
|
||||
|
30
src/test/run-pass/macro-nt-list.rs
Normal file
30
src/test/run-pass/macro-nt-list.rs
Normal file
@ -0,0 +1,30 @@
|
||||
// Copyright 2014 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.
|
||||
|
||||
#![feature(macro_rules)]
|
||||
|
||||
macro_rules! list (
|
||||
( ($($id:ident),*) ) => (());
|
||||
( [$($id:ident),*] ) => (());
|
||||
( {$($id:ident),*} ) => (());
|
||||
)
|
||||
|
||||
macro_rules! tt_list (
|
||||
( ($($tt:tt),*) ) => (());
|
||||
)
|
||||
|
||||
pub fn main() {
|
||||
list!( () );
|
||||
list!( [] );
|
||||
list!( {} );
|
||||
|
||||
tt_list!( (a, b, c) );
|
||||
tt_list!( () );
|
||||
}
|
Loading…
Reference in New Issue
Block a user