Don't leak a tmp_length variable in the global namespace #6

Merged
merged 2 commits into from Mar 26, 2013

Projects

None yet

2 participants

@strk
Contributor
strk commented Mar 22, 2013

Leak was detected by mocha

@nmrugg
nmrugg commented on cd8c37a Mar 23, 2013

Thanks for pointing out that mistake; however, after reviewing the code, I realized that the real problem is that I misspelled the variable on line 1080. If you want to write a new patch, I'll pull that.

Owner
strk replied Mar 25, 2013

I've pushed a second commit in the existing pull request. You can pull --squash to get a single commit out of it

@nmrugg nmrugg merged commit 69c4d8d into LZMA-JS:master Mar 26, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment