Skip to content

(RFC) replace old_iter::repeat with the Times trait #6106

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

Closed
wants to merge 1 commit into from
Closed

(RFC) replace old_iter::repeat with the Times trait #6106

wants to merge 1 commit into from

Conversation

thestinger
Copy link
Contributor

I don't have a strong opinion on the function vs. method, but there's no point in having both. I'd like to make a repeat adaptor like Python/Haskell for turning a value into an infinite stream of the value, so this has to at least be renamed.

@catamorphism
Copy link
Contributor

I'm not sure how I feel about this change; sometimes repeat seems to make more sense to me, other times the n.times syntax makes more sense to me. (Though as a functional programmer, I usually use the former.)

I suggested that @thestinger make this an "RFC" since it's a change the affects a lot of existing code.

@kud1ing
Copy link

kud1ing commented May 13, 2013

See also #2143

@bstrie
Copy link
Contributor

bstrie commented May 19, 2013

Unified function call syntax would make the "function vs method" issue moot, so I'm fine accepting this for now.

bors added a commit that referenced this pull request May 19, 2013
I don't have a strong opinion on the function vs. method, but there's no point in having both. I'd like to make a `repeat` adaptor like Python/Haskell for turning a value into an infinite stream of the value, so this has to at least be renamed.
@bors bors closed this May 19, 2013
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.

5 participants