Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

macros: tt fragments treat $(foo)* like foo #39390

Closed
jseyfried opened this issue Jan 29, 2017 · 1 comment · Fixed by #39419
Closed

macros: tt fragments treat $(foo)* like foo #39390

jseyfried opened this issue Jan 29, 2017 · 1 comment · Fixed by #39419
Assignees
@jseyfried
Labels
A-macros Area: All kinds of macros (custom derive, macro_rules!, proc macros, ..)

Comments

@jseyfried
Copy link
Contributor

jseyfried commented Jan 29, 2017 •
edited
Loading

For example, the following compiles and prints success:

macro_rules! identity {
    ($t:tt) => { $t }
}

identity! {
    $(fn $(main)* () {
        $($( println!($("success")+); )+)+
    })*
}
@jseyfried jseyfried self-assigned this Jan 29, 2017
@jseyfried jseyfried added the A-macros Area: All kinds of macros (custom derive, macro_rules!, proc macros, ..) label Jan 29, 2017
@jseyfried
Copy link
Contributor Author

cc @nrc

@jseyfried jseyfried mentioned this issue Jan 31, 2017
Merged
@jseyfried jseyfried mentioned this issue Feb 11, 2017
Closed
frewsxcv added a commit to frewsxcv/rust that referenced this issue Mar 1, 2017
@frewsxcv
Rollup merge of rust-lang#39419 - jseyfried:simplify_tokentree, r=nrc
4ba49ab 
Simplify `TokenTree` and fix `macro_rules!` bugs

This PR
 - fixes rust-lang#39390, fixes rust-lang#39403, and fixes rust-lang#39404 (each is a [breaking-change], see issues for examples),
 - fixes rust-lang#39889,
 - simplifies and optimizes macro invocation parsing,
 - cleans up `ext::tt::transcribe`,
 - removes `tokenstream::TokenTree::Sequence` and `Token::MatchNt`,
   - instead, adds a new type `ext::tt::quoted::TokenTree` for use by `macro_rules!` (`ext::tt`)
 - removes `parser.quote_depth` and `parser.parsing_token_tree`, and
 - removes `quote_matcher!`.
   - Instead, use `quote_tokens!` and `ext::tt::quoted::parse` the result with `expect_matchers=true`.
   - I found no outside uses of `quote_matcher!` when searching Rust code on Github.

r? @nrc
bors added a commit that referenced this issue Mar 1, 2017
@bors
Auto merge of #39419 - jseyfried:simplify_tokentree, r=nrc
7ce1fbe 
Simplify `TokenTree` and fix `macro_rules!` bugs

This PR
 - fixes #39390, fixes #39403, and fixes #39404 (each is a [breaking-change], see issues for examples),
 - fixes #39889,
 - simplifies and optimizes macro invocation parsing,
 - cleans up `ext::tt::transcribe`,
 - removes `tokenstream::TokenTree::Sequence` and `Token::MatchNt`,
   - instead, adds a new type `ext::tt::quoted::TokenTree` for use by `macro_rules!` (`ext::tt`)
 - removes `parser.quote_depth` and `parser.parsing_token_tree`, and
 - removes `quote_matcher!`.
   - Instead, use `quote_tokens!` and `ext::tt::quoted::parse` the result with `expect_matchers=true`.
   - I found no outside uses of `quote_matcher!` when searching Rust code on Github.

r? @nrc
jseyfried added a commit to jseyfried/rust that referenced this issue Mar 10, 2017
@jseyfried
Fix rust-lang#39390 on beta.
70b6f8b
@jseyfried jseyfried mentioned this issue Mar 10, 2017
Closed
brson pushed a commit to brson/rust that referenced this issue Mar 10, 2017
@jseyfried @brson
Fix rust-lang#39390 on beta.
eac7c8e
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jseyfried jseyfried

Labels
A-macros Area: All kinds of macros (custom derive, macro_rules!, proc macros, ..)
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Simplify TokenTree and fix macro_rules! bugs jseyfried/rust
1 participant
@jseyfried