<html>
<body>
<div style="font-family: Verdana, Arial, Helvetica, Sans-Serif;">
<table bgcolor="#f9f3c9" width="100%" cellpadding="8" style="border: 1px #c9c399 solid;">
<tr>
<td>
This is an automatically generated e-mail. To reply, visit:
<a href="http://git.reviewboard.kde.org/r/111553/">http://git.reviewboard.kde.org/r/111553/</a>
</td>
</tr>
</table>
<br />
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">I tested a video with srt, txt and sub file extensions. And I found srt worked fine. But when I played a video with .sub as subtitle file then it was failing show the subtitle. So may be you may have to look into the sub files. Since I don't have txt files. I couldn't test them. </pre>
<br />
<p>- Sujith</p>
<br />
<p>On July 17th, 2013, 5:25 p.m. UTC, Akshay Ratan wrote:</p>
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('http://git.reviewboard.kde.org/static/rb/images/review_request_box_top_bg.ab6f3b1072c9.png'); background-position: left top; background-repeat: repeat-x; border: 1px black solid;">
<tr>
<td>
<div>Review request for Plasma, Shantanu Tushar, Sinny Kumari, and Sujith Haridasan.</div>
<div>By Akshay Ratan.</div>
<p style="color: grey;"><i>Updated July 17, 2013, 5:25 p.m.</i></p>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Description </h1>
<table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: 1px solid #b8b5a0">
<tr>
<td>
<pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">This patch essentially aims at extending the subtitle feature to .sub file format also.
Also, this serves as a fix after the Revision 2f4ab1e6 (https://projects.kde.org/projects/extragear/multimedia/plasma-mediacenter/repository/revisions/2f4ab1e61149bd9b9487d985845ebdc1b8ec41bb/diff/libs/mediacenter/subtitleprovider.cpp) breaks the existing .srt subtitle support as well.
Steps to test:
1) Play a movie or any video file which has a .sub or .txt or .srt format subtitle.
2) Subtitles should come as the video play ( Ofcourse only is the subtitle file name is same as that of movie file name)
</pre>
</td>
</tr>
</table>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Testing </h1>
<table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: 1px solid #b8b5a0">
<tr>
<td>
<pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Shantanu : I guess the subtitle patch which got committed in the master in that revision was by mistake. That broke the existing subtitle support for .srt as well. Applying this one over it solves the issue :)
However, please test it once as still on my system , when a .sub subtitle should play , following is the debug console error output ::
(plasma-mediacenter:12999): GStreamer-CRITICAL **: gst_pad_link_full: assertion `GST_IS_PAD (sinkpad)' failed
.srt subtitle plays fine :)
</pre>
</td>
</tr>
</table>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Diffs</b> </h1>
<ul style="margin-left: 3em; padding-left: 0;">
<li>libs/mediacenter/subtitleprovider.h <span style="color: grey">(8647ded)</span></li>
<li>libs/mediacenter/subtitleprovider.cpp <span style="color: grey">(32094b0)</span></li>
<li>mediaelements/mediaplayer/MediaPlayer.qml <span style="color: grey">(2567312)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/111553/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>