wxianfeng / hanzi_to_pinyin_node

chinese hanzi to pinyin via node.js
http://wxianfeng.com/2013/08/31/node-js-hanzi-to-pinyin
MIT License
10 stars 0 forks source link

unnecessary global variable usage. #3

Closed hyss777 closed 7 years ago

hyss777 commented 7 years ago

你好,我在使用这个库,首先表示感谢。

lib/hanzi_to_pinyin.js第42行的i忘记加var,导致暴露i为全局变量,我在npm的包里做了手工修改,建议此包作者能把这个var加上。

wxianfeng commented 7 years ago

Fixed, 3x