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

Utilize Result::unwrap_err in more places. #33474

Merged
merged 1 commit into from
May 10, 2016
Merged

Utilize Result::unwrap_err in more places. #33474

merged 1 commit into from
May 10, 2016

Conversation

frewsxcv
Copy link
Member

@frewsxcv frewsxcv commented May 6, 2016

No description provided.

@frewsxcv frewsxcv changed the title Utilize Option::unwrap_err in more places. Utilize Result::unwrap_err in more places. May 6, 2016
@frewsxcv frewsxcv force-pushed the unwrap-err branch 2 times, most recently from f7d1db7 to 3df7720 Compare May 7, 2016 17:11
@alexcrichton
Copy link
Member

@bors: r+ 3df7720

@Manishearth Manishearth mentioned this pull request May 8, 2016
Closed
Manishearth added a commit to Manishearth/rust that referenced this pull request May 8, 2016
@Manishearth
Rollup merge of rust-lang#33474 - frewsxcv:unwrap-err, r=alexcrichton
82fa821 
Utilize `Result::unwrap_err` in more places.

None
bors added a commit that referenced this pull request May 8, 2016
@bors
Auto merge of #33501 - Manishearth:rollup, r=Manishearth
6d369a8 
Rollup of 9 pull requests

- Successful merges: #32900, #33129, #33365, #33383, #33474, #33478, #33480, #33484, #33493
- Failed merges: #33360
@bors
Copy link
Contributor

bors commented May 9, 2016

⌛ Testing commit 3df7720 with merge 8b11c61...

bors added a commit that referenced this pull request May 9, 2016
@bors
Auto merge of #33474 - frewsxcv:unwrap-err, r=alexcrichton
8b11c61 
Utilize `Result::unwrap_err` in more places.

None
@bors
Copy link
Contributor

bors commented May 9, 2016

💔 Test failed - auto-linux-64-opt

@frewsxcv
Copy link
Member Author

frewsxcv commented May 9, 2016

Fixed that one issue. Tests pass on Travis now.

@alexcrichton
Copy link
Member

@bors: r+ 62b19c6

Manishearth added a commit to Manishearth/rust that referenced this pull request May 9, 2016
@Manishearth
Rollup merge of rust-lang#33474 - frewsxcv:unwrap-err, r=alexcrichton
476029f 
Utilize `Result::unwrap_err` in more places.

None
Manishearth added a commit to Manishearth/rust that referenced this pull request May 9, 2016
@Manishearth
Rollup merge of rust-lang#33474 - frewsxcv:unwrap-err, r=alexcrichton
cfe8259 
Utilize `Result::unwrap_err` in more places.

None
@Manishearth Manishearth mentioned this pull request May 9, 2016
Merged
bors added a commit that referenced this pull request May 9, 2016
@bors
Auto merge of #33523 - Manishearth:rollup, r=Manishearth
9d68267 
Rollup of 10 pull requests

- Successful merges: #33129, #33224, #33370, #33383, #33431, #33474, #33480, #33496, #33509, #33514
- Failed merges:
Manishearth added a commit to Manishearth/rust that referenced this pull request May 9, 2016
@Manishearth
Rollup merge of rust-lang#33474 - frewsxcv:unwrap-err, r=alexcrichton
fffaf66 
Utilize `Result::unwrap_err` in more places.

None
bors added a commit that referenced this pull request May 9, 2016
@bors
Auto merge of #33523 - Manishearth:rollup, r=Manishearth
e0fd34b 
Rollup of 10 pull requests

- Successful merges: #33129, #33224, #33370, #33383, #33431, #33474, #33480, #33496, #33509, #33514
- Failed merges:
@bors bors merged commit 62b19c6 into rust-lang:master May 10, 2016
@frewsxcv frewsxcv deleted the unwrap-err branch May 10, 2016 01:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@frewsxcv @alexcrichton @bors