Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix overlapping subtitles #7719

Merged
merged 2 commits into from Aug 5, 2015
Merged

fix overlapping subtitles #7719

merged 2 commits into from Aug 5, 2015

Conversation

FernetMenta
Copy link
Contributor

@stefansaraev
Copy link
Contributor

thanks! I could reproduce the bug in livetv (vdr + vnsi) and this fixes it.

jenkins build this please

@Jalle19
Copy link
Member

Jalle19 commented Aug 5, 2015

Noticed this yesterday as well, I thought it was the broadcasters fault.

+1

FernetMenta added a commit that referenced this pull request Aug 5, 2015
@FernetMenta FernetMenta merged commit 9628f60 into xbmc:master Aug 5, 2015
@FernetMenta FernetMenta deleted the subfix branch August 5, 2015 14:23
@stefansaraev
Copy link
Contributor

should this be backported ?

@FernetMenta
Copy link
Contributor Author

I am not 100% certain that it does not break anything. On the other hand it fixes a bug. At least it should be tested properly, whatever this means :) I did sone quick tests with SSA, DVB, and text based subs.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Backport: Done Type: Fix non-breaking change which fixes an issue
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants