Review Request: Add a new KDBusService class to manage the D-Bus registration

Kevin Ottens ervin at kde.org
Sat Jul 9 16:43:39 BST 2011


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/101902/
-----------------------------------------------------------

Review request for kdelibs and David Faure.


Summary
-------

This class is meant to get the D-Bus registration logic out of KApplication, so that it can also be easily provided to QCoreApplication. It also allows to easily provide the KUniqueApplication behavior, which is tested with the accompanying unit test.

Note that it's a somewhat early review request, I'm aware that the APIDOX is missing for instance. It's just that it'll likely be a generally useful class, so I'd like to gather feedback early. In short, just in case you were wondering why it seems unfinished: it is unfinished. :-)


Diffs
-----

  kdecore/CMakeLists.txt 4d96b50 
  kdecore/kernel/kdbusservice.h PRE-CREATION 
  kdecore/kernel/kdbusservice.cpp PRE-CREATION 
  kdecore/tests/CMakeLists.txt 7c3db77 
  kdecore/tests/kdbusservicetest.cpp PRE-CREATION 

Diff: http://git.reviewboard.kde.org/r/101902/diff


Testing
-------


Thanks,

Kevin

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20110709/48e77a67/attachment.htm>


More information about the kde-core-devel mailing list