<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/110301/">http://git.reviewboard.kde.org/r/110301/</a>
</td>
</tr>
</table>
<br />
<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 Nepomuk and Vishesh Handa.</div>
<div>By Simeon Bird.</div>
<p style="color: grey;"><i>Updated May 4, 2013, 7:11 p.m.</i></p>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Changes</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;">Make watchFolder not accessible.</pre>
</td>
</tr>
</table>
<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;">Filewatch: move all filewatching into the filewatch service
Remove watch adding in the fileindexer service.
On a config change they were previously being re-added twice.
Then watch all indexed folders in filewatch constructor.
This is needed so that nepomukctl restart filewatch watches all indexed folders.
Don't bother to wait two minutes before installing the watches - it
isn't that slow anymore and we were doing most of the work anyway by
watching the homedir on startup.</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;">Compiled, ran, checked it responds to config changes correctly.</pre>
</td>
</tr>
</table>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Diffs</b> (updated)</h1>
<ul style="margin-left: 3em; padding-left: 0;">
<li>interfaces/org.kde.nepomuk.FileWatch.xml <span style="color: grey">(0befe7c476140d257e183f91a5e0cfed142b7eef)</span></li>
<li>services/fileindexer/fileindexer.h <span style="color: grey">(1171b1190768f89491c66b6c4540f6631ef3b4cc)</span></li>
<li>services/fileindexer/fileindexer.cpp <span style="color: grey">(bd7ab6b83ced4cd02a4ffd20e85f64e52ac9acfd)</span></li>
<li>services/filewatch/nepomukfilewatch.h <span style="color: grey">(52057df5ab3c8174c07c5542251c8c4f6bc65fdc)</span></li>
<li>services/filewatch/nepomukfilewatch.cpp <span style="color: grey">(c3e72931a218e9afc757cc3bcf51e14fc9e3689d)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/110301/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>