danmakuC icon indicating copy to clipboard operation
danmakuC copied to clipboard

支持多条的正则表达式, 支持弹幕合并,

Open My-Responsitories opened this issue 1 year ago • 6 comments

close #13, close #14

  • 支持由\n分割的多条的正则表达式, 但对utf8字符的处理仍有问题, std和boost的regex都不能很好的处理utf8
  • 支持弹幕合并
  • 简单优化了一下代码

My-Responsitories avatar Feb 14 '24 06:02 My-Responsitories

感谢贡献,如果引入了新的boost文件,需要相应进行添加,不然CI会失败

HFrost0 avatar Feb 18 '24 08:02 HFrost0

如果C++内部不好做正则可以考虑在python中进行过滤,应该不至于影响太多性能

HFrost0 avatar Feb 18 '24 08:02 HFrost0

感谢贡献,如果引入了新的boost文件,需要相应进行添加,不然CI会失败

boost的正则对utf8的处理也不好, 所以没加, 用的还是std 不过 https://github.com/google/re2 倒是可以正常处理utf8字符, 但我不会写CI

My-Responsitories avatar Feb 19 '24 02:02 My-Responsitories

如果C++内部不好做正则可以考虑在python中进行过滤,应该不至于影响太多性能

弹幕合并是在正则过滤前进行的, 在python里过滤会把相同内容的弹幕反复判断

My-Responsitories avatar Feb 19 '24 02:02 My-Responsitories

CI我来看看吧,没有加入新的文件应该就没有问题。自测没有问题后可以merge

HFrost0 avatar Feb 19 '24 07:02 HFrost0

有进展吗?没计划我可以新开一个 PR

sherlcok314159 avatar Mar 07 '24 03:03 sherlcok314159