vavr icon indicating copy to clipboard operation
vavr copied to clipboard

Published 20 hours ago • verified publisher icon vavr-io

Iterator should internally not make use of toStream() anymore

Open danieldietrich opened this issue 8 years ago • 3 comments

Instead we should implement stateful Iterators that achieve the same functionality. The reason is memory-efficiency.

  • [ ] foldRight(Object, BiFunction)
  • [ ] partition(Predicate)
  • [ ] reduceRight(BiFunction)
  • [ ] slideBy(Function)
  • [ ] span(Predicate)
  • [ ] unfoldLeft(Object, Function)
  • [ ] unzip(Function)
  • [ ] unzip3(Function)

danieldietrich avatar Apr 12 '17 08:04 danieldietrich

Closing the issue, Iterator now does not contain toStream() anymore. Thx @ruslansennov for fixing it!

danieldietrich avatar Jun 29 '17 20:06 danieldietrich

Will re-open it, also Stream.ofAll(...) should not be used anymore by Iterator internally.

danieldietrich avatar Jun 30 '17 07:06 danieldietrich

Note: Iterator does not extends Traversable anymore. It might be an option not to implement all methods.

danieldietrich avatar Jan 08 '19 07:01 danieldietrich