<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/104480/">http://git.reviewboard.kde.org/r/104480/</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;">If this works for Last.fm this should to. I do remember problems with last.fm when the wallet was not available though, so I wonder if all corner cases are covered.

There are also so style errors in the copied code: there should be no space between if and '('

if( bla )

{

}

I think the copyright header is ok. You probably don't even need the "from" info, but in this case it's useful because there is no git trace.</pre>

 <br />

<p>- Bart</p>

<br />

<p>On April 3rd, 2012, 8:04 p.m., Andrzej Hunt 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 Andrzej Hunt.</div>

<p style="color: grey;"><i>Updated April 3, 2012, 8:04 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;">Modifies the Magnatune service to use KWallet for storage (instead of plaintext).

The code has been copied from LastFmServiceConfig.cpp, with almost no modification on my part. (KDevelop removed some whitespace and reformatted some other bits of code meaning there are some formatting changes in the diff -- I apologise if this is a problem for reviewing, and I can revert the formatting changes if desired.)

A question about copyright attribution:  Would this be the correct thing to add to MagnatuneConfig.cpp below the current copyright line?

 * Code copied from ../lastfm/LastFmServiceConfig.cpp:

 * Copyright (c) 2007 Shane King <kde@dontletsstart.com>                                

 * Copyright (c) 2009 Leo Franchi <lfranchi@kde.org> </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=242256">242256</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/services/magnatune/MagnatuneConfig.cpp <span style="color: grey">(18ee898)</span></li>

 <li>src/services/magnatune/MagnatuneConfig.h <span style="color: grey">(f1d25eb)</span></li>

</ul>

<p><a href="http://git.reviewboard.kde.org/r/104480/diff/" style="margin-left: 3em;">View Diff</a></p>

  </td>

 </tr>

</table>

  </div>

 </body>

</html>