Properly timed WebVTT/TTML now broken
@fcartegnie said:
I've mentioned many times that the only way to set an offset conditionally was to check if the timings were overlapping with video
@typx said:
Or to let the user enable the retiming or not with an option.
Conditional offset would probably require a new demux Control similar to DEMUX_SET_NEXT_DEMUX_TIME (as in specific to slave demuxes) to provide the master's normal time and length, check for the overlap and if it overlaps return in DEMUX_GET_NORMAL_TIME the value provided by this new control.
(Or do it in the core in which case we need a new control to get the first and last pts+duration. Fun fact if the subtitles are properly timed DEMUX_GET_LENGTH is already mismatched with the TS's length since it doesn't substract the first pts)
But it could also fail for many reasons so we probably need an option to disable this anyway.
Regression from !3853 (merged)