<table><tr><td style="">narvaez added inline comments.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D19007">View Revision</a></tr></table><br /><div><strong>INLINE COMMENTS</strong><div><div style="margin: 6px 0 12px 0;"><div style="border: 1px solid #C7CCD9; border-radius: 3px;"><div style="padding: 0; background: #F7F7F7; border-color: #e3e4e8; border-style: solid; border-width: 0 0 1px 0; margin: 0;"><div style="color: #74777d; background: #eff2f4; padding: 6px 8px; overflow: hidden;"><a style="float: right; text-decoration: none;" href="https://phabricator.kde.org/D19007#inline-105986">View Inline</a><span style="color: #4b4d51; font-weight: bold;">bruns</span> wrote in <span style="color: #4b4d51; font-weight: bold;">monitor.cpp:90</span></div>
<div style="margin: 8px 0; padding: 0 12px; color: #74777D;"><p style="padding: 0; margin: 8px;">The batchsize 40 by default, so currently the update interval is 200 files. 100 or 200 does not matter to much, it is just a crude limiter anyway.</p>

<p style="padding: 0; margin: 8px;">A better fix would be emit the remaining time directly from the scheduler, which could adapt much better to fast and slow progress. And listening to signals on DBus is in general preferable to querying.</p></div></div>
<div style="margin: 8px 0; padding: 0 12px;"><p style="padding: 0; margin: 8px;">The batch size is <strong>hardcoded</strong> at 40 so querying it at at any interval is pointless, for that matter. Instead of hardcoding more magic numbers I would just #define BATCH_SIZE 40 inside the Engine lib and set the update interval to 5 * BATCH_SIZE.</p></div></div></div></div></div><br /><div><strong>REPOSITORY</strong><div><div>R293 Baloo</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D19007">https://phabricator.kde.org/D19007</a></div></div><br /><div><strong>To: </strong>narvaez, Baloo, bruns, astippich, poboiko<br /><strong>Cc: </strong>davidedmundson, kde-frameworks-devel, Baloo, ashaposhnikov, michaelh, astippich, spoorun, ngraham, bruns, abrahams<br /></div>