Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

Unnecessary usage of the new operator #68

Merged
merged 1 commit into from Sep 29, 2012

Conversation

Projects
None yet
2 participants
Contributor

tzbob commented Sep 26, 2012

A tokenizer object is created on the next line, the Tokenizer variable should be a pure function with a filled prototype and not another object.

I was using browserify to get client-sided phonetics and stumbled upon this mistake. Browserify failed on embedding the aggressive tokenizer, fixing this mistake also fixed the browserifying of phonetics.

fixed typo, should not create a new object of the required tokenizer …
…as this is done on the next line. This also fixes browserifying the phonetic section
Owner

chrisumbel commented Sep 29, 2012

Thanks

chrisumbel added a commit that referenced this pull request Sep 29, 2012

Merge pull request #68 from Tzbob/master
Unnecessary usage of the new operator

@chrisumbel chrisumbel merged commit c5f32d8 into NaturalNode:master Sep 29, 2012

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