stream::repeat_item

[stepancheg]

Procuces the same cloned element forever.

stream::repeat name would mimic iter::repeat, however,
stream::repeat with Result param is not possible usually, because
most error types (e. g. io::Error) are not Clone. Hence the
name repeat_item.

[alexcrichton]

I suspect that 99% of usage of this function would be to repeat something akin to Ok(T), so I wouldn't mind calling this repeat. We could even add repeat_err later if we really wanted to, but I suspect it won't ever come to that.

[Stebalien]

Hmm... before posting an RFC issue, do you think a From<T> for Result<T, E> impl would be useful? This seems like something that has come up several times in futures-rs discussions. Unfortunately, the fact that default types for generics don't influence inference may make less useful in cases like this.

[stepancheg]

Renamed repeat_item to repeat.

[alexcrichton]

Could this be called Repeat instead? I see that IterStream doesn't follow this convention but I think that may be a mistake.

[stepancheg]

Renamed RepeatStream to Repeat.

[alexcrichton]

@Stebalien perhaps, but let's not discuss that here.

@stepancheg thanks!