search

Masaiki / xy-VSFilter

xy-VSFilter variant with libass backend
https://github.com/Masaiki/xy-VSFilter
GNU General Public License v2.0
543 stars 10 forks source link

Fix incorrect display of subtitle track list when using mpc-hc as player #22

Closed pkuislm closed 1 year ago

pkuislm commented 1 year ago

本来我是在mpc-hc那里改的，今天早上起来我又想了想，觉得还是在xySubFilter这里改可能好点。理由如下：

  1. 我觉得clsid2说的没错，内置的字幕流mpc-hc已经选择好了，xySubFilter只能拿到选好的那一条。这样的话，干脆就把那一条由mpc-hc提供的名称写死。
  2. 外置字幕那边，CSimpleTextSubtitle的m_name处理的似乎有问题，为了能把字幕名称完整地显示出来，我对CSimpleTextSubtitle::Open做了点改动。现在m_name存储的是字幕文件名，至于XySubFilter::get_LanguageName，我直接让他在拿到文件名之后再找有没有表示字幕语言的部分了。为了能在XySubFilter::Info里拿到m_name，我还在XySubFilter::GetIsEmbeddedSubStream下面加了一个新函数：XySubFilter::GetSubStreamName（不过我感觉这个函数加的不是很妥当，或许有更好的解决方法？）

最终效果如下： image

不知道这样的改动可不可行呢？

Masaiki commented 1 year ago

感谢 PR，工作日没时间看，周末会看的

pkuislm commented 1 year ago

感谢 PR，工作日没时间看，周末会看的

收到，感谢回复。工作辛苦，您先忙

Masaiki commented 1 year ago

总体上没什么问题，辛苦了

Masaiki commented 1 year ago

Close https://github.com/Masaiki/xy-VSFilter/issues/16

pkuislm commented 1 year ago

总体上没什么问题，辛苦了

ok，已修正出现的问题