Global leaks #38

Closed
Sannis opened this Issue Jul 26, 2012 · 7 comments

Projects

None yet

3 participants

@Sannis
Sannis commented Jul 26, 2012

There is at least two leaks -in- under Node.js: encodeHTML and def.

Refs tj/consolidate.js#46.

@Sannis
Sannis commented Jul 26, 2012

Update: I may fix this and add tests (are you ok with mocha?), if you consider this as bug.

@tj
tj commented Jul 26, 2012

what? those are leaks in dot.js

@tj
tj commented Jul 26, 2012

fixed the hamljs one, for some reason that global was not showing up for me

@Sannis
Sannis commented Jul 26, 2012

I mean "leak in doT under Node.js", thanks for clarification.

@tj
tj commented Jul 26, 2012

oh wow sorry im stupid i didn't realize this was in the doT repo hahahhaha...

@olado
Owner
olado commented Dec 24, 2012

Version 1.0.0 fixes this.

@olado olado closed this Dec 24, 2012
@olado olado reopened this Dec 24, 2012
@Sannis
Sannis commented Feb 9, 2013

Tnx!

@Sannis Sannis closed this Feb 9, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment