Skip to content
This repository was archived by the owner on Sep 5, 2018. It is now read-only.

Commit a40a323

Browse files
committed
Merge pull request #2 from DarkDimius/patch-3
Add open questions about zip and signature of `flatMap`
2 parents 4473179 + c8c16ad commit a40a323

File tree

1 file changed

+7
-0
lines changed

1 file changed

+7
-0
lines changed

text/0000-collection-view-redesign.md

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -767,6 +767,13 @@ This proposal leaves open for discussion/contribution several things:
767767
parallel collections.
768768
3. Additional performance analysis/consideration for Transducer implementations
769769
not in the prototype.
770+
4. Should `zip` force either of collections? It it worth it?
771+
5. What should be the signature of `flatMap`? should the argument function return `TraversableOnce` or a `View`?
772+
[here](https://github.com/axel-angel/scala-blitzview/blob/master/src/main/scala/scala/collection/view/BlitzViews.scala#L21) @DarkDimius has a solution that allows to support both:
773+
774+
```scala
775+
def flatMap[C, U](f: El => U)(implicit ctx: Scheduler, viewable: BlitzView.IsViewable[U, C]): BlitzView[C]
776+
```
770777

771778
## References
772779

0 commit comments

Comments
 (0)