Skip to content

Commit

Permalink
Allow concat_idents! in type positions as well as in expression pos…
Browse files Browse the repository at this point in the history
…itions
  • Loading branch information
jseyfried committed May 19, 2016
1 parent 088d417 commit e992794
Show file tree
Hide file tree
Showing 2 changed files with 38 additions and 21 deletions.
50 changes: 32 additions & 18 deletions src/libsyntax_ext/concat_idents.rs
Original file line number Diff line number Diff line change
Expand Up @@ -52,22 +52,36 @@ pub fn expand_syntax_ext<'cx>(cx: &mut ExtCtxt, sp: Span, tts: &[TokenTree])
}
let res = str_to_ident(&res_str);

let e = P(ast::Expr {
id: ast::DUMMY_NODE_ID,
node: ast::ExprKind::Path(None,
ast::Path {
span: sp,
global: false,
segments: vec!(
ast::PathSegment {
identifier: res,
parameters: ast::PathParameters::none(),
}
)
}
),
span: sp,
attrs: None,
});
MacEager::expr(e)
struct Result { ident: ast::Ident, span: Span };

impl Result {
fn path(&self) -> ast::Path {
let segment = ast::PathSegment {
identifier: self.ident,
parameters: ast::PathParameters::none()
};
ast::Path { span: self.span, global: false, segments: vec![segment] }
}
}

impl base::MacResult for Result {
fn make_expr(self: Box<Self>) -> Option<P<ast::Expr>> {
Some(P(ast::Expr {
id: ast::DUMMY_NODE_ID,
node: ast::ExprKind::Path(None, self.path()),
span: self.span,
attrs: None,
}))
}

fn make_ty(self: Box<Self>) -> Option<P<ast::Ty>> {
Some(P(ast::Ty {
id: ast::DUMMY_NODE_ID,
node: ast::TyKind::Path(None, self.path()),
span: self.span,
}))
}
}

Box::new(Result { ident: res, span: sp })
}
9 changes: 6 additions & 3 deletions src/test/compile-fail/syntax-extension-minor.rs
Original file line number Diff line number Diff line change
Expand Up @@ -8,12 +8,15 @@
// option. This file may not be copied, modified, or distributed
// except according to those terms.

// this now fails (correctly, I claim) because hygiene prevents
// the assembled identifier from being a reference to the binding.
#![feature(concat_idents)]
#![feature(concat_idents, type_macros)]

pub fn main() {
struct Foo;
let _: concat_idents!(F, oo) = Foo; // Test that `concat_idents!` can be used in type positions

let asdf_fdsa = "<.<".to_string();
// this now fails (correctly, I claim) because hygiene prevents
// the assembled identifier from being a reference to the binding.
assert!(concat_idents!(asd, f_f, dsa) == "<.<".to_string());
//~^ ERROR: unresolved name `asdf_fdsa`

Expand Down

0 comments on commit e992794

Please sign in to comment.