Skip to content

Conversation

qnighy
Copy link
Contributor

@qnighy qnighy commented Jun 7, 2017

This patch just replaces matches with ? in the compiler, which I came across when I'm reading the parser.

@rust-highfive
Copy link
Contributor

r? @nikomatsakis

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

@eddyb
Copy link
Member

eddyb commented Jun 7, 2017

@bors r+ rollup

@bors
Copy link
Collaborator

bors commented Jun 7, 2017

📌 Commit ab72611 has been approved by eddyb

@aidanhs aidanhs added the S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. label Jun 7, 2017
frewsxcv added a commit to frewsxcv/rust that referenced this pull request Jun 8, 2017
…, r=eddyb

Replace some matches with try.

This patch just replaces `match`es with `?` in the compiler, which I came across when I'm reading the parser.
bors added a commit that referenced this pull request Jun 8, 2017
Rollup of 5 pull requests

- Successful merges: #42470, #42490, #42497, #42510, #42512
- Failed merges:
@bors bors merged commit ab72611 into rust-lang:master Jun 8, 2017
@qnighy qnighy deleted the just-use-try-in-three-places branch June 30, 2017 16:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants