Skip to content

Rewrite ttml conversion#2957

Merged
TobiGr merged 6 commits intoTeamNewPipe:devfrom
kapodamy:subtitles
Jan 26, 2020
Merged

Rewrite ttml conversion#2957
TobiGr merged 6 commits intoTeamNewPipe:devfrom
kapodamy:subtitles

Conversation

@kapodamy
Copy link
Contributor

rewrite SubtitleConverter (use JSoup library instead, remove unused methods)
fixes #2891

rewrite SubtitleConverter (use JSoup library instead, remove unused methods)
@TobiGr TobiGr added downloader Issue is related to the downloader bug Issue is related to a bug labels Jan 14, 2020
* add missing namespace of StandardCharsets
* use an unused constructor argument
@TobiGr TobiGr added this to the 0.18.2 milestone Jan 18, 2020
Copy link
Contributor

@TobiGr TobiGr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me. Just two minor things

@TobiGr TobiGr merged commit bf5e94f into TeamNewPipe:dev Jan 26, 2020
@ghost ghost mentioned this pull request Jan 28, 2020
2 tasks
@kapodamy kapodamy deleted the subtitles branch February 15, 2020 18:24
This was referenced Feb 19, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Issue is related to a bug downloader Issue is related to the downloader

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Newpipe download subtitle (.srt) without line breaks

2 participants