Skip to content

Fixed comment and refactored functions (libstd/option.rs) #12888

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Closed
wants to merge 2 commits into from
Closed

Fixed comment and refactored functions (libstd/option.rs) #12888

wants to merge 2 commits into from

Conversation

aochagavia
Copy link
Contributor

The old comment of as_mut_slice() did not describe the function correctly. The new one does.

Also refactored option::iter() and option::mut_iter() to use as_ref() and as_mut() instead of match.

The old comment did not describe the function correctly
@alexcrichton
Copy link
Member

This plus #12889 and #12890 look good to me, but I'll wait to r+ until those other two PRs are merged into this one (they can still be separate commits)

Replaced match by self.as_ref() and self.as_mut()
@aochagavia
Copy link
Contributor Author

Done

bors added a commit that referenced this pull request Mar 14, 2014
The old comment of as_mut_slice() did not describe the function correctly. The new one does.

Also refactored option::iter() and option::mut_iter() to use as_ref() and as_mut() instead of match.
@bors bors closed this Mar 15, 2014
@aochagavia aochagavia deleted the Fix-comment branch May 13, 2014 06:43
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants