Skip BOM, if exists, when probing file format

chenxuuu commented 8 years ago

增加检查BOM项 fix the BOM bug

m13253 commented 8 years ago

谢谢你的改进。我希望指出两点事情，第一，这个参数可以是一个文件名，也可以是一个已经打开的文件对象，还可以是一个网络流。我承认我当年写这个的时候没有写好注释，抱歉让你费解。希望能处理一下。其次，希望能把代码的排版通过 PEP8 效验：Tab 还是空格，PEP8 有建议性规定，与其抱怨不如配置文本编辑器把 Tab 按钮设置成自动输入4个空格。

我个人建议判断逻辑在 ProbeCommentFormat 函数里面做。因为这个函数有两个 decorator，能保证不会因为 EOF 而崩溃，结束的时候会 seek 回 0。大致做法是，判断是否是某个网站的弹幕格式之前，先跳过 BOM。如果成功判断了弹幕格式是什么网站的，后面的解析就不需要管 BOM 了。

十分抱歉我在复习考试，不能亲自改。所以请你改好了跟我说一下，麻烦你啦~

chenxuuu commented 8 years ago

@m13253 四个空格那是我只会basic和c对的原因。。。吐个槽而已、、、至少py基本语句还是懂的。。

m13253 / danmaku2ass

Skip BOM, if exists, when probing file format #25