Skip to content

Conversation

@jdonaldson
Copy link

Here's something else that I think will be useful. It's a bit different from your other "Lazy" class in that it only deals with iterators. But, the theme is the same.

@jdonaldson
Copy link
Author

One other option is to have all these methods defined statically. You could get the same flow here with "using".

@kevinresol
Copy link
Member

Reminds me of this again haxetink/tink_pure#1

@jdonaldson
Copy link
Author

Yeah, that looks like the same idea, I figured you guys had something like this.

One small note, I see that you're using Sequence as an iterable. Wouldn't it be better to handle lazy operations solely within iterators? It would give more flexibility (e.g. it would work on iterators that cannot be repeated). A more specialized Sequence type could be built on top of it.

@back2dos
Copy link
Member

Closing in favor of the other PR. This lib is getting big enough already ^^

@back2dos back2dos closed this Aug 14, 2020
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.

3 participants