Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Alternate versions of fileTape and runDrive #4

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

erantapaa
Copy link
Contributor

This is an example of an alternate way to write the fileTape and runDrive functions which avoids the dummy respond Unit call in fileTape. You also might be able to remove the need for Unit in the TResp type.

Example usage:

example = runDrive' (fileTape' "some-path") driver

See also this thread in the pipes mailing list:

https://groups.google.com/d/msg/haskell-pipes/qK5c61mALKI/9miuMIijAQAJ

@YoEight
Copy link
Owner

YoEight commented Aug 20, 2015

Thank you ! Why not making this version the real one ?

@erantapaa
Copy link
Contributor Author

Ok - I've replaced the old versions of fileTape and runDrive with the new ones.

I'm a little confused by the type of the new runDrive - perhaps you can figure that out.

@YoEight
Copy link
Owner

YoEight commented Sep 8, 2015

Sorry I was in vacations, I'll see what I can do here

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.

2 participants