Skip to content
New issue

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

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

修正逻辑错误 #43

Merged
merged 1 commit into from
Sep 3, 2015
Merged

修正逻辑错误 #43

merged 1 commit into from
Sep 3, 2015

Commits on Sep 3, 2015

  1. 修正逻辑错误， 此处逻辑应该为 先验证是否stime < time 再验证此条弹幕是否能通过filter的验证， 否则当补全validat… 

    …e方法之后会产生让人抓狂的bug。

bug 具体内容： 当我实现完整的validate方法之后， 若按照原来的逻辑， 会执行else 的语句， 但是else 中直接break, 所以并未执行for的第三句话。 所以导致this.position一直未自增
    @jabbany @dbFlower
    jabbany authored and dbFlower committed Sep 3, 2015
    Configuration menu
    Copy the full SHA
    6150321 View commit details
    Browse the repository at this point in the history