elbart / node-memcache

node.js memcached client
285 stars 64 forks source link

NPM package not updated since variable leak fix four months ago #32

Open DanielSmedegaardBuus opened 11 years ago

DanielSmedegaardBuus commented 11 years ago

Hi :)

The fix for the global variable ("line") leak from four months ago hasn't been submitted to the NPM repository. It's still at January 2012, including the small bug (throws an error when Mocha testing).

Thanks :)

pacmac commented 11 years ago

Yes, I am encountering this memory leak right now !

xsurge83 commented 11 years ago

my unit test detected a global leak on line 340 in memcache.js

readLine() needs a var or function readline(string)