D10078: Add separate lib KF5::DBusRunner

Friedrich W. H. Kossebau noreply at phabricator.kde.org
Thu Apr 26 15:31:07 UTC 2018


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
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20180426/fd1cfcc7/attachment.html>


More information about the Kde-frameworks-devel mailing list