search

terasum / js-mdict

*.mdx/*.mdd interpreter js implements, support mdict index file
MIT License
159 stars 18 forks source link

fix(mdict.js): correct property name #33

Closed songxiaocheng closed 3 years ago

terasum commented 3 years ago

@songxiaocheng 请问这么改的原因是什么呀？

songxiaocheng commented 3 years ago

@songxiaocheng 请问这么改的原因是什么呀？

在我 IDE 里，keyBlockDecompAccumulator 和 keyBlockDecompSize 处直接就提示 Unresolved variable 的。这里当 i + 1 >= list.length 时候会报错，https://github.com/terasum/js-mdict/blob/0852fe63400869f17f0899377a3839597b86bc7d/src/mdict.js#L70-L76

因为赋值的时候用的就是如下的名字 https://github.com/terasum/js-mdict/blob/dca08bf9489cf5b3e5090b87926d100cb99e3c77/src/mdict-base.js#L847-L855

而且，这个列表是叫 record block info list， 但是里面的属性如果用 key block 做前缀，好像不太对劲？

还有，就是好像其他函数也有这个property名字不对。

songxiaocheng commented 3 years ago

又更新了下 parse_defination 中的这两个属性。

terasum commented 3 years ago

@songxiaocheng 这个问题我还需要花点时间再看下哈