D21606: RFC: ThreadWeaver Job Decorators not used properly and have no effect

fvogt abandoned this revision. fvogt added a comment. D22758 got merged REPOSITORY R308 KRunner REVISION DETAIL https://phabricator.kde.org/D21606 To: fvogt Cc: apol, davidedmundson, kde-frameworks-devel, LeGast00n, GB_2, michaelh, ngraham, bruns

D21606: RFC: ThreadWeaver Job Decorators not used properly and have no effect

apol added a comment. I need this additionally diff --git a/src/runnermanager.cpp b/src/runnermanager.cpp index f92830d..6ec0c3c 100644 --- a/src/runnermanager.cpp +++ b/src/runnermanager.cpp @@ -345,6 +345,7 @@ public: emit

D21606: RFC: ThreadWeaver Job Decorators not used properly and have no effect

davidedmundson added a comment. https://bugs.kde.org/show_bug.cgi?id=381835 REPOSITORY R308 KRunner REVISION DETAIL https://phabricator.kde.org/D21606 To: fvogt Cc: davidedmundson, kde-frameworks-devel, LeGast00n, michaelh, ngraham, bruns

D21606: RFC: ThreadWeaver Job Decorators not used properly and have no effect

fvogt created this revision. Herald added a project: Frameworks. Herald added a subscriber: kde-frameworks-devel. fvogt requested review of this revision. REVISION SUMMARY ThreadWeaver Job Decorators don't actually hook into an existing job. Instead, they provide the same interface as the job