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