<html>
<body>
<div style="font-family: Verdana, Arial, Helvetica, Sans-Serif;">
<table bgcolor="#f9f3c9" width="100%" cellpadding="12" style="border: 1px #c9c399 solid; border-radius: 6px; -moz-border-radius: 6px; -webkit-border-radius: 6px;">
<tr>
<td>
This is an automatically generated e-mail. To reply, visit:
<a href="https://git.reviewboard.kde.org/r/129205/">https://git.reviewboard.kde.org/r/129205/</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;"><p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">I don't know about this... It smells like an "eliphant approach" for something seemingly simple (you just want to know which files/folders changed if you had done a search).</p>
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">I don't know what the right approach is, so i'm going to reason a bit here. David will probably reply and come up with an awesome solution ;)</p>
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Approach 1:
I think (assumption) that the ioslave itself should monitor for changes and react accordingly.
However, for that to work the slave itself needs to notify the one that called the slave and shout: "Hey, i have new changed data! Here it is.". But there doesn't seem to be an approach for that in the save interface. It spits out listEntry calls when you open it and ends with a finished() call, there is no (or is there but am i missing it somehow?) way for an ioslave to emit new entries after finished() has been called.
If you want that, then you probably have to extend the slave logic to allow for subsequent entries after finished is called.
Also, i'm quite sure that inotify (the notify backend that you likely use on linux) spits out all the information you need to know already. The interfaces that KDE exposes as wrappers don't emit as much as inotify does. They mainly seem to be making somewhat sure that the different notify backends behave the same.</p>
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Approach 2:
You can go (and you did) go to "the client side" by letting KDirLister(and friends) handle this for you. It's possible, but then you lose the low level stuff you had availible in the ioslave. You are fully at the mercy of KDirWatch(and friends) to make that work as you want. Performance wise, this is probably a bad idea.</p></pre>
<br />
<p>- Mark Gaiser</p>
<br />
<p>On oktober 24th, 2016, 7:05 a.m. UTC, Anthony Fieroni wrote:</p>
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="12" style="border: 1px #888a85 solid; border-radius: 6px; -moz-border-radius: 6px; -webkit-border-radius: 6px;">
<tr>
<td>
<div>Review request for KDE Frameworks and David Faure.</div>
<div>By Anthony Fieroni.</div>
<p style="color: grey;"><i>Updated okt 24, 2016, 7:05 a.m.</i></p>
<div style="margin-top: 1.5em;">
<b style="color: #575012; font-size: 10pt;">Repository: </b>
kio
</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;"><p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">David, i will discard review if you don't like it, cause watching files changes can be <em style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: normal;">really</em> expensive. I try to:
1. to not break abi compability
2. to extend filenamesearch with this option
3. to fix https://git.reviewboard.kde.org/r/129141/</p></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;"><p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">For 3. i still can't figure out why in filenamesearch signal for delete item(s) is not triggered.
1. Search for file (by name) in dolphin
2. When appear in view delete him
3. Signal ItemsDeleted is not triggered, file stays in the view even if new search is performed and reload is needed. The cache look good, tests pass, works but in filenamesearch.</p></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/core/kcoredirlister.h <span style="color: grey">(e6ba2ac)</span></li>
<li>src/core/kcoredirlister.cpp <span style="color: grey">(508516e)</span></li>
<li>src/core/kcoredirlister_p.h <span style="color: grey">(9a3cc7b)</span></li>
<li>tests/kdirlistertest_gui.h <span style="color: grey">(8316b20)</span></li>
<li>tests/kdirlistertest_gui.cpp <span style="color: grey">(11e89cf)</span></li>
</ul>
<p><a href="https://git.reviewboard.kde.org/r/129205/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>