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.

Sign up for GitHub

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

Jump to bottom

Fix Streaming#memoization. #665

Merged
merged 4 commits into from
Nov 19, 2015
Merged

Fix Streaming#memoization. #665

merged 4 commits into from
Nov 19, 2015

Commits on Nov 16, 2015

  1. Fix Streaming#memoization. 

    This commit fixes a bug noticed by @ceedubs. The basic issue
is that we have to be sure to memoize the Eval instance holding
the tail while *also* memoizing the tail itself when it is
calculated. Previously we were only memoizing the head node.
    @non
    non committed Nov 16, 2015
    Configuration menu
    Copy the full SHA
    d55f559 View commit details
    Browse the repository at this point in the history

  2. Improve comment.

    @non
    non committed Nov 16, 2015
    Configuration menu
    Copy the full SHA
    34edb5d View commit details
    Browse the repository at this point in the history

Commits on Nov 19, 2015

  1. Merge remote-tracking branch 'origin/master' into bug/fix-memoize

    @non
    non committed Nov 19, 2015
    Configuration menu
    Copy the full SHA
    36ef6af View commit details
    Browse the repository at this point in the history

  2. Removed unused code from streaming test.

    @non
    non committed Nov 19, 2015
    Configuration menu
    Copy the full SHA
    8ed18f5 View commit details
    Browse the repository at this point in the history