ngraham accepted this revision. ngraham added a comment. This revision is now accepted and ready to land.
Works for me and the code looks sane. Looks like this fixes https://bugs.kde.org/show_bug.cgi?id=364858, so can you add `BUG: 364858` and `FIXED-IN: 5.52` to the summary section and also make the changes that @bruns requested? Thanks! Also, please don't commit this until after Frameworks 5.51 has been tagged (which is tomorrow). This is the kind of change that we should have a month to test out ourselves before releasing it to customers. REPOSITORY R293 Baloo BRANCH monitor-state (branched from master) REVISION DETAIL https://phabricator.kde.org/D15943 To: poboiko, #frameworks, #baloo, ngraham Cc: ngraham, bruns, kde-frameworks-devel, ashaposhnikov, michaelh, astippich, spoorun, abrahams
