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

Docs: Change "statements" to "expressions" on match #32443

Merged
merged 1 commit into from
Mar 23, 2016

Conversation

tclfs
Copy link
Contributor

@tclfs tclfs commented Mar 23, 2016

I apt to use expressions over statements, because match is a expression in essence, though it can become a statement when followed a semicolon.

I apt to use `expressions` over `statements`, because `match` is a expression in essence, though it can become a statement when followed a semicolon.
@rust-highfive
Copy link
Collaborator

r? @Manishearth

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

@steveklabnik
Copy link
Member

@bors: r+ rollup

Agreed, thanks!

On Mar 23, 2016, 00:05 -0400, Rust highfive [email protected], wrote:

r?@Manishearth(https://github.com/manishearth)

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


You are receiving this because you are subscribed to this thread.
Reply to this email directly orview it on GitHub(#32443 (comment))

@bors
Copy link
Contributor

bors commented Mar 23, 2016

📌 Commit ea9ae48 has been approved by steveklabnik

eddyb added a commit to eddyb/rust that referenced this pull request Mar 23, 2016
Docs: Change "statements" to "expressions" on `match`

I apt to use `expressions` over `statements`, because `match` is a expression in essence, though it can become a statement when followed a semicolon.
bors added a commit that referenced this pull request Mar 23, 2016
Rollup of 11 pull requests

- Successful merges: #32404, #32420, #32423, #32425, #32429, #32430, #32431, #32434, #32437, #32441, #32443
- Failed merges:
@bors bors merged commit ea9ae48 into rust-lang:master Mar 23, 2016
@tclfs tclfs deleted the patch-8 branch March 24, 2016 02:45
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