Skip to content

Switch option::t to option everywhere #1719

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
Closed
marijnh opened this issue Jan 31, 2012 · 4 comments
Closed

Switch option::t to option everywhere #1719

marijnh opened this issue Jan 31, 2012 · 4 comments
Labels
C-cleanup Category: PRs that clean code up or issues documenting cleanup. E-easy Call for participation: Easy difficulty. Experience needed to fix: Not much. Good first issue.
Milestone
0.2 release, deta...

Comments

@marijnh
Copy link
Contributor

marijnh commented Jan 31, 2012

I just noticed that the core::option type is finally working. Using it will make things look quite a bit better.
@graydon
Copy link
Contributor

graydon commented Jan 31, 2012

Yes please. Bumping to 0.2 blocking. It's cleanup for a rainy day but this sort of mess ought to be nipped in the bud.

@ghost ghost assigned catamorphism Feb 1, 2012
@catamorphism
Copy link
Contributor

Brain is fried, so I'll take this.

@nikomatsakis
Copy link
Contributor

+1

@catamorphism
Copy link
Contributor

Fixed as of 49cb3fc

@prasannavl prasannavl mentioned this issue Oct 17, 2018
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
C-cleanup Category: PRs that clean code up or issues documenting cleanup. E-easy Call for participation: Easy difficulty. Experience needed to fix: Not much. Good first issue.
Projects
None yet
Milestone
0.2 release, detail work, cleanup and...
Development

No branches or pull requests
4 participants
@graydon @marijnh @nikomatsakis @catamorphism