Skip to content

reference: fix definition of :tt#38163

Merged
bors merged 1 commit into
rust-lang:masterfrom
durka:patch-33
Dec 7, 2016
Merged

reference: fix definition of :tt#38163
bors merged 1 commit into
rust-lang:masterfrom
durka:patch-33

Conversation

@durka

@durka durka commented Dec 4, 2016

Copy link
Copy Markdown
Contributor

The reference says that $x:tt matches "either side of the => in macro_rules` which is technically true but completely uninformative. This changes that bullet point to what the book says (a single token or sequence of token trees inside brackets).

@rust-highfive

Copy link
Copy Markdown
Contributor

r? @steveklabnik

(rust_highfive has picked a reviewer for you, use r? to override)

The reference says that $x:tt matches "either side of the `=>` in macro_rules` which is technically true but completely uninformative. This changes that bullet point to what the book says (a single token or sequence of token trees inside brackets).
@bluss

bluss commented Dec 4, 2016

Copy link
Copy Markdown
Contributor

@bors r+ rollup

@bors

bors commented Dec 4, 2016

Copy link
Copy Markdown
Collaborator

📌 Commit dbfcdd4 has been approved by bluss

@bluss

bluss commented Dec 4, 2016

Copy link
Copy Markdown
Contributor

thanks!

GuillaumeGomez added a commit to GuillaumeGomez/rust that referenced this pull request Dec 7, 2016
reference: fix definition of :tt

The reference says that $x:tt matches "either side of the `=>` in macro_rules` which is technically true but completely uninformative. This changes that bullet point to what the book says (a single token or sequence of token trees inside brackets).
bors added a commit that referenced this pull request Dec 7, 2016
Rollup of 9 pull requests

- Successful merges: #38085, #38123, #38151, #38153, #38158, #38163, #38186, #38189, #38208
- Failed merges:
@bors bors merged commit dbfcdd4 into rust-lang:master Dec 7, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants