kossebau updated this revision to Diff 33152. kossebau added a comment.
- make MatchReply a QObject & change isValid to isFinished (+ signal) + allows to make MatchReplyPrivate unaware of AbstractRunnerPrivate by using the signal for unregistering + closer in API terms to existing *Reply classes in Qt spheres + finished signal can be used in eventloop-based processing to cancel - cancel for now any still active matchreply once a new match request arrives removes the need for implementors to decide how to deal with overlapping requests now and in the future, they just can rely on the finished state of the MatchReply object to decide whether to complete it REPOSITORY R308 KRunner CHANGES SINCE LAST UPDATE https://phabricator.kde.org/D10078?vs=33114&id=33152 BRANCH kdbusrunnerlib2 REVISION DETAIL https://phabricator.kde.org/D10078 AFFECTED FILES CMakeLists.txt KF5DBusRunnerConfig.cmake.in autotests/CMakeLists.txt autotests/dbusrunnertest.cpp autotests/testremoteasyncrunner.cpp autotests/testremoteasyncrunner.h autotests/testremoterunner.cpp autotests/testremoterunner.h src/CMakeLists.txt src/dbusrunner/CMakeLists.txt src/dbusrunner/abstractrunner.cpp src/dbusrunner/abstractrunner.h src/dbusrunner/abstractrunner_p.cpp src/dbusrunner/abstractrunner_p.h src/dbusrunner/action.h src/dbusrunner/dbusrunner1adaptor.cpp src/dbusrunner/dbusrunner1adaptor_p.h src/dbusrunner/matchreply.cpp src/dbusrunner/matchreply.h src/dbusrunner/matchreply_p.h src/dbusrunner/querymatch.h src/querymatch.h To: kossebau, broulik, davidedmundson Cc: bruns, michaelh, ngraham, #frameworks
