<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/106175/">http://git.reviewboard.kde.org/r/106175/</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;">This review has been submitted with commit 8c7c3c9082a31687c508cf44096b4687e7f8cde9 by Sam Lade to branch master.</pre>
<br />
<p>- Commit</p>
<br />
<p>On August 25th, 2012, 12:51 p.m., Milan Izai wrote:</p>
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('http://git.reviewboard.kde.org/media/rb/images/review_request_box_top_bg.png'); background-position: left top; background-repeat: repeat-x; border: 1px black solid;">
<tr>
<td>
<div>Review request for Amarok.</div>
<div>By Milan Izai.</div>
<p style="color: grey;"><i>Updated Aug. 25, 2012, 12:51 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;">After finishing the last track in "Repeat Playlist" mode, Amarok scans playlist from the beginning in order to determine the next track to play. However the last track played is not included in this scan. This leads to a strange behavior when playlist contains only 1 track. In this case Amarok just stops playback instead of repeating the only track in playlist. The bugfix is trivial: just include the last track when scanning playlist.</pre>
</td>
</tr>
</table>
<div style="margin-top: 1.5em;">
<b style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Bugs: </b>
<a href="https://bugs.kde.org/show_bug.cgi?id=290955">290955</a>
</div>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Diffs</b> </h1>
<ul style="margin-left: 3em; padding-left: 0;">
<li>src/playlist/navigators/StandardTrackNavigator.cpp <span style="color: grey">(f9af2ef8687bc691bcce419f4773d6038db6c9af)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/106175/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>