Fixes for memory issues #6

Merged
merged 1 commit into from Mar 23, 2011

Conversation

Projects
None yet
2 participants
Contributor

draggor commented Mar 23, 2011

I did a bad thing and I blew away my old fork, made a new one to get your rand fixes, and then put my stuff back in. This one turns the c++ module into a class, and now the JS part merely extends extends it. All tests pass without needing any changes, other than removing the destroy() tests, as they're no longer necessary.

@substack substack merged commit 439a8b3 into substack:master Mar 23, 2011

Owner

substack commented Mar 23, 2011

Awesome stuff! This just landed as bigint v0.3. I especially like that destroy() is no longer necessary.

Contributor

draggor commented Mar 23, 2011

Woohoo! The only thing you might not like is I accidentally ended up
putting back all those tabs.

On Wed, Mar 23, 2011 at 6:06 PM, substack
reply@reply.github.com
wrote:

Awesome stuff! This just landed as bigint v0.3. I especially like that destroy() is no longer necessary.

Reply to this email directly or view it on GitHub:
#6 (comment)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment