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

Improve doc for Result::unwrap() #24606

Merged
merged 1 commit into from
Apr 20, 2015
Merged

Improve doc for Result::unwrap() #24606

merged 1 commit into from
Apr 20, 2015

Conversation

bombless
Copy link
Contributor

Get rid of the confusion that what does "custom" mean in this context.

@rust-highfive
Copy link
Collaborator

r? @brson

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

@bombless
Copy link
Contributor Author

r? @steveklabnik

@rust-highfive rust-highfive assigned steveklabnik and unassigned brson Apr 19, 2015
@bombless
Copy link
Contributor Author

Ah, my fault, I think @alexcrichton would be more familiar with this code.
r? @alexcrichton

@bombless
Copy link
Contributor Author

I just switched to the wrong PR, I was intended to change the reviewer of another PR :(
nvm

Former commit broke the 80-characters rule, fixed.

@alexcrichton
Copy link
Member

@bors: r+ c3c7610

@alexcrichton
Copy link
Member

Thanks!

bors added a commit that referenced this pull request Apr 20, 2015
Get rid of the confusion that what does "custom" mean in this context.
@bors
Copy link
Contributor

bors commented Apr 20, 2015

⌛ Testing commit c3c7610 with merge f46c4e1...

@bors bors merged commit c3c7610 into rust-lang:master Apr 20, 2015
@steveklabnik steveklabnik mentioned this pull request Apr 25, 2015
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.

6 participants