Skip to content

Conversation

@dkrieg
Copy link

@dkrieg dkrieg commented May 23, 2014

Please consider this contribution.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should we be trying to make functions total and return Option<B> here?

@mperry
Copy link
Contributor

mperry commented Jun 24, 2015

I created some similar, but better classes to implement the stream processing chapter from the FP in Scala book, see https://github.com/mperry/stream-processing/tree/master/src/main/java/com/github/mperry/fj. This is an improvement and was useful to implement the chapter, but not sure if it is useful enough to include. I will close this PR for now and we can reconsider this if people have opinions they'd like to share.

@mperry mperry closed this Jun 24, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants