- Moderator
- #21
But i can understand Tourette point that some .TS has subtitle teletext inside the stream, that say teletext sub for me on CSAT package is broken so ....(but that another story).
I think therefore it is nice to have the option.
But i can understand Tourette point that some .TS has subtitle teletext inside the stream, that say teletext sub for me on CSAT package is broken so ....(but that another story).
Yes, it is unfortunate if this patch breaks teletext subtitle support for .ts files in My Videos. Perhaps we only need to use another splitter if there is external subtitles stored with the .ts file?
No need for an extra Configuration option in this case However this might not be a good idea if there's other things than subtitles to consider...
- Single .ts file -> Use TSReader
- .ts and .srt files (for example) -> Use system default splitter (probably LAV or even TSReader)
Since we are talking about a change to the way MP operates, we should have a good reason to change the behaviour (hence the discussion).
I agree that the video player should be consistent, so it should use the splitter selection set by the user (in Config), and we default this to LAV at present.
So we could either have another selection pop-up (just for .ts splitter selection), or for .ts files only use the splitter filter merit - if we make the TsReader.ax merit very low then the 'selected' splitter would be used, but anyone (like me or tourettes) who wants to use TsReader can just set the merit high enough to make it the preferred splitter filter.
Default option will be off (so no change) user will not see any change but if user want to use the new option, he can turn it on..
I continue my regression testing