<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/105276/">http://git.reviewboard.kde.org/r/105276/</a>
</td>
</tr>
</table>
<br />
<div>
<table width="100%" border="0" bgcolor="white" style="border: 1px solid #C0C0C0; border-collapse: collapse; margin: 2px padding: 2px;">
<thead>
<tr>
<th colspan="4" bgcolor="#F0F0F0" style="border-bottom: 1px solid #C0C0C0; font-size: 9pt; padding: 4px 8px; text-align: left;">
<a href="http://git.reviewboard.kde.org/r/105276/diff/1/?file=68125#file68125line148" style="color: black; font-weight: bold; text-decoration: underline;">src/services/lastfm/meta/LastFmMeta_p.h</a>
<span style="font-weight: normal;">
(Diff revision 1)
</span>
</th>
</tr>
</thead>
<tbody style="background-color: #e4d9cb; padding: 4px 8px; text-align: center;">
<tr>
<td colspan="4"><pre style="font-size: 8pt; line-height: 140%; margin: 0; ">class Track::Private : public QObject</pre></td>
</tr>
</tbody>
<tbody>
<tr>
<th bgcolor="#b1ebb0" style="border-right: 1px solid #C0C0C0;" align="right"><font size="2"></font></th>
<td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; "></pre></td>
<th bgcolor="#b1ebb0" style="border-left: 1px solid #C0C0C0; border-right: 1px solid #C0C0C0;" align="right"><font size="2">148</font></th>
<td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; "> <span class="k">if</span><span class="p">(</span> <span class="o">!</span><span class="n">lfm</span><span class="p">.</span><span class="n">parse</span><span class="p">(</span> <span class="n">m_userFetch</span><span class="o">-></span><span class="n">readAll</span><span class="p">()</span> <span class="p">)</span> <span class="p">)</span></pre></td>
</tr>
</tbody>
</table>
<pre style="margin-left: 2em; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">a small change would improve consistency here, change the code to:
if( lfm.parse(...) )
{
do stuff
}
else
{
debug
}
(like all the other error handling is done)</pre>
</div>
<br />
<p>- Stefan</p>
<br />
<p>On June 16th, 2012, 10:50 p.m., Harald Sitter 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 Harald Sitter.</div>
<p style="color: grey;"><i>Updated June 16, 2012, 10:50 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;">port to liblastfm1, mind that this is slightly api incompatible with liblastfm0, and retaining ifdef compat does not seem worthwhile given the fact that latter uses exceptions and former does not</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;">[~] login test in settings (for some reason the service does not pick up changes there until restart, which seems actually architectural as the KCM does not trigger reinitialization of the service)
[+] streaming
[+] love
[+] ban
[+] next
[+] friends/neighbors
[+] upcoming events plasmoid
[+] related artists plasmoid
</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>src/services/ampache/LastfmInfoParser.cpp <span style="color: grey">(50cb529516ce1358785fa04824cd32642a526495)</span></li>
<li>src/services/lastfm/LastFmService.cpp <span style="color: grey">(a14d989b3fac81e545df095adee36580b052fea7)</span></li>
<li>src/services/lastfm/LastFmServiceCollection.cpp <span style="color: grey">(22f49ec0fe8e9f917f8a26f9d546ad147ad04e5b)</span></li>
<li>src/services/lastfm/LastFmServiceSettings.cpp <span style="color: grey">(2eff71e51dc30605b4d395323a01e24b8fa25d1e)</span></li>
<li>src/services/lastfm/LastFmTreeModel.h <span style="color: grey">(ac171a57a36ec6bb5443035b11bea77e4ae94720)</span></li>
<li>src/services/lastfm/LastFmTreeModel.cpp <span style="color: grey">(2d94d67c5fe10bfc1abba3dd9848fd1177daa518)</span></li>
<li>src/services/lastfm/ScrobblerAdapter.h <span style="color: grey">(67ed73b291cef61ee37b0ddae28ba1eadfd6304d)</span></li>
<li>src/services/lastfm/biases/LastFmBias.cpp <span style="color: grey">(b011b3299f98d851109d3d9ba52e6fd09855dcff)</span></li>
<li>src/services/lastfm/biases/WeeklyTopBias.cpp <span style="color: grey">(df6dd6681f05d95a0e281d6e82224de2bbf001e6)</span></li>
<li>src/services/lastfm/meta/LastFmMeta.h <span style="color: grey">(f7b93c14c92c0bebf99d165656a0e34f8b991cc0)</span></li>
<li>src/services/lastfm/meta/LastFmMeta.cpp <span style="color: grey">(d8f52845cb801d8ae940bc1bca77c7281ef18379)</span></li>
<li>src/services/lastfm/meta/LastFmMeta_p.h <span style="color: grey">(c2d88ac79a4dd0fe59b9f889baf72c879833f0d3)</span></li>
<li>src/services/lastfm/meta/MultiPlayableCapabilityImpl_p.h <span style="color: grey">(1c464cc2635c7d614eab48ca5e4e156c67870f04)</span></li>
<li>src/context/engines/upcomingevents/UpcomingEventsEngine.cpp <span style="color: grey">(4ae73d892cef3d4a41662f7ed15849ae773c16e3)</span></li>
<li>src/core/support/Amarok.h <span style="color: grey">(c9852154dee74fb372bae435fdeefa92003d41b1)</span></li>
<li>CMakeLists.txt <span style="color: grey">(0d146c58bd382b4d4f379ebedddf1630d73db369)</span></li>
<li>cmake/modules/FindLibLastFm.cmake <span style="color: grey">(1d863b08489e27054a049e35d2d94667646db1f7)</span></li>
<li>src/LastfmReadLabelCapability.cpp <span style="color: grey">(385e6b9c065009b330e116e3d748b82c24f14e8d)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/105276/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>