<table><tr><td style="">bruns created this revision.<br />bruns added reviewers: Baloo, mgallien, astippich, ngraham.<br />Herald added projects: Frameworks, Baloo.<br />Herald added a subscriber: kde-frameworks-devel.<br />bruns requested review of this revision.
</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/D28980">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>Revert "add missing license"<br />
This reverts commit <a href="https://phabricator.kde.org/R293:96fb5896c304b7292fc3cea87fb584b14b270448" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">96fb5896c304b7292fc3cea87fb584b14b270448</a>.</p>

<p>This reverts commit <a href="https://phabricator.kde.org/R293:ef0e268c657718d4a353638f91d3723624c358ac" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">ef0e268c657718d4a353638f91d3723624c358ac</a>.</p>

<p>The BalooWatcherApplication DBus interface was created to be used by<br />
Elisa, but the only use there is to create some debug output.<br />
As soon as Elisa registers itself, each rename causes a bunch<br />
of DB queries (in case a directory is renamed, it builds the list<br />
of all children) and a (potentially huge) DBus call.</p>

<p>The DBus interface is totally awkward, as it calls a method in the<br />
watcher application (i.e. replicating the message on the sender side)<br />
instead of using DBus signals and letting the DBus daemon do its job.</p>

<p>Keep the DBus interface as a stub, but remove the caused overhead.</p></div></div><br /><div><strong>TEST PLAN</strong><div><p>ctest</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R293 Baloo</div></div></div><br /><div><strong>BRANCH</strong><div><div>config_rework</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D28980">https://phabricator.kde.org/D28980</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>autotests/unit/file/CMakeLists.txt<br />
autotests/unit/file/filewatchtest.cpp<br />
autotests/unit/file/metadatamovertest.cpp<br />
autotests/unit/file/metadatamovertest.h<br />
src/engine/transaction.cpp<br />
src/engine/transaction.h<br />
src/file/CMakeLists.txt<br />
src/file/filewatch.cpp<br />
src/file/filewatch.h<br />
src/file/mainhub.cpp<br />
src/file/mainhub.h<br />
src/file/metadatamover.cpp<br />
src/file/metadatamover.h</div></div></div><br /><div><strong>To: </strong>bruns, Baloo, mgallien, astippich, ngraham<br /><strong>Cc: </strong>kde-frameworks-devel, hurikhan77, lots0logs, LeGast00n, cblack, fbampaloukas, domson, ashaposhnikov, michaelh, astippich, spoorun, ngraham, bruns, abrahams<br /></div>