Review Request 127383: Avoid introspecting a DBus interface when we don't use it.

David Edmundson david at davidedmundson.co.uk
Wed Mar 16 01:21:03 UTC 2016


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

Review request for KDE Frameworks.


Repository: kdbusaddons


Description
-------

QDBusInterface is only useful if you want to know at runtime what methods a remote
interface has.

Using the generated interfaces skips the initial query to
.DBus.Introspect(). This makes everything 1 roundtrip faster.

Realistically it won't have any actual impact, but IMHO it's a bit cleaner


Diffs
-----

  src/CMakeLists.txt 6da110c405f348a1105870e7527e0b67646224ac 
  src/kdbusservice.cpp 30c485c9f52bcd361f29cc419b75c0c6215901eb 

Diff: https://git.reviewboard.kde.org/r/127383/diff/


Testing
-------

started kate twice, got a second windows with expected file
started konversation twice, first instance got reactivated.


Thanks,

David Edmundson

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20160316/41de3db4/attachment.html>


More information about the Kde-frameworks-devel mailing list