<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/100960/">http://git.reviewboard.kde.org/r/100960/</a>
     </td>
    </tr>
   </table>
   <br />


<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('http://git.reviewboard.kde.orgrb/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 kdelibs and Frederik Gladhorn.</div>
<div>By Matthias Fuchs.</div>





<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;">Thus insuring that the cache is always consistent.
This way it can't happen, that e.g. DownloadManager updates some KNS3::Entries and a call to DownloadWidget later on would overwrite the updated ones as updateable.</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;">Tested with the patch in https://git.reviewboard.kde.org/r/100958/ since that fixes updateing.

Test case:
Implemented auto-updating for the comic plamsoid.
After some updates happen the user clicks the get more button to install an additional comic. In the old behaviour the comic would show up as not installed after restarting, as the cache was not shared [1]. With the new behaviour any updated comic will show up as installed and the newly installed comic will also show up as installed.

[1] Here it only depended which cache was written last to the disk. So either all comics would still show up as updatedable or the one comic would show up as not installed.</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>knewstuff/knewstuff3/core/cache.h <span style="color: grey">(5b0b60e)</span></li>

 <li>knewstuff/knewstuff3/core/cache.cpp <span style="color: grey">(53aa55e)</span></li>

 <li>knewstuff/knewstuff3/core/engine.h <span style="color: grey">(5aa7449)</span></li>

 <li>knewstuff/knewstuff3/core/engine.cpp <span style="color: grey">(1017628)</span></li>

</ul>

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




  </td>
 </tr>
</table>




  </div>
 </body>
</html>