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.

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

doc: concat_idents! macro: more on its limitations. #31542

Merged
merged 1 commit into from
Feb 14, 2016

Conversation

nodakai
Copy link
Contributor

@nodakai nodakai commented Feb 10, 2016

Just a small documentation change.

It would be great if anyone could check my English.

@rust-highfive
Copy link
Collaborator

r? @aturon

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

/// capture local variables, and macros are only allowed in item,
/// statement or expression position, meaning this macro may be difficult to
/// use in some situations.
/// capture local variables. Also, as a general rule, macros are only
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

one space after a period please!

@steveklabnik
Copy link
Member

Your English is great! Just a small nit about the spaces and let's :shipit:

Signed-off-by: NODA, Kai <nodakai@gmail.com>
@nodakai
Copy link
Contributor Author

nodakai commented Feb 10, 2016

Thanks, corrected to a single space style.

@steveklabnik
Copy link
Member

@bors: r+ rollup

On Feb 10, 2016, 15:26 -0500, Kai Nodanotifications@github.com, wrote:

Thanks, corrected to a single space style.


Reply to this email directly orview it on GitHub(#31542 (comment)).

@bors
Copy link
Contributor

bors commented Feb 10, 2016

📌 Commit cec158b has been approved by steveklabnik

steveklabnik added a commit to steveklabnik/rust that referenced this pull request Feb 13, 2016
…klabnik

Just a small documentation change.

It would be great if anyone could check my English.
Manishearth added a commit to Manishearth/rust that referenced this pull request Feb 13, 2016
…klabnik

Just a small documentation change.

It would be great if anyone could check my English.
bors added a commit that referenced this pull request Feb 13, 2016
@bors bors merged commit cec158b into rust-lang:master Feb 14, 2016
@nodakai nodakai deleted the concat_idents-desc branch March 8, 2016 05:03
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