Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with
or
.
Download ZIP

Loading…

Global leaks #74

Merged
merged 1 commit into from

2 participants

@camshaft

There were some global leaks in a few of the tokenizers so I went ahead and fixed them

@chrisumbel
Owner

thank you much!

@chrisumbel chrisumbel merged commit f703081 into NaturalNode:master
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Commits on Nov 6, 2012
  1. @camshaft

    Fixed global leaks

    camshaft authored
This page is out of date. Refresh to see the latest.
View
2  lib/natural/phonetics/phonetic.js
@@ -21,7 +21,7 @@ THE SOFTWARE.
*/
var stopwords = require('../util/stopwords');
-var Tokenizer = require('../tokenizers/aggressive_tokenizer')
+var Tokenizer = require('../tokenizers/aggressive_tokenizer'),
tokenizer = new Tokenizer();
module.exports = function() {
View
2  lib/natural/tokenizers/aggressive_tokenizer_fa.js
@@ -24,7 +24,7 @@ THE SOFTWARE.
var Tokenizer = require('./tokenizer'),
util = require('util');
-AggressiveTokenizer = function() {
+var AggressiveTokenizer = function() {
Tokenizer.call(this);
};
util.inherits(AggressiveTokenizer, Tokenizer);
View
4 spec/aggressive_tokenizer_spec.js
@@ -20,8 +20,8 @@ OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN
THE SOFTWARE.
*/
-var Tokenizer = require('lib/natural/tokenizers/aggressive_tokenizer')
- tokenizer = new Tokenizer();
+var Tokenizer = require('lib/natural/tokenizers/aggressive_tokenizer'),
+ tokenizer = new Tokenizer();
describe('aggressive_tokenizer', function() {
it('should tokenize strings', function() {
Something went wrong with that request. Please try again.