Skip to content

Alternate take at fixing P1.memo cache issues #108 and #106 - #127

Closed
jbgi wants to merge 1 commit into
masterfrom
unknown repository
Closed

jbgi wants to merge 1 commit into
masterfrom
unknown repository

Conversation

@jbgi

@jbgi jbgi commented Apr 14, 2015

Copy link
Copy Markdown
Member

avoid repetitive locks in case of null returned by delegate.

avoid repetitive locks in case of null return by delegate.
@mperry

mperry commented May 28, 2015

Copy link
Copy Markdown
Contributor

I have created PR #141 which takes your 3 state byte variable and uses an Option for the same purpose. I think this is closer to the original and how we use the library, but they look essentially the same.

@mperry mperry changed the title Alternate take at fixing #108 and #106 Alternate take at fixing P1.memo cache issues #108 and #106 May 28, 2015
@jbgi

jbgi commented May 28, 2015

Copy link
Copy Markdown
Member Author

superseded by #141.

@jbgi jbgi closed this May 28, 2015
@mperry mperry added the invalid label May 30, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants