This revision was automatically updated to reflect the committed changes.
Closed by commit R289:1c97e1d9741f: Don't block starting notification service
(authored by davidedmundson).
REPOSITORY
R289 KNotifications
CHANGES SINCE LAST UPDATE
https://phabricator.kde.org/D6963?vs=18912=18933
vandenoever added a comment.
The patch looks clean, but I do not fully understand how is supposed to work.
Perhaps an explanation of the different scenarios would help future readers of
this code.
REPOSITORY
R289 KNotifications
BRANCH
master
REVISION DETAIL
mart accepted this revision.
This revision is now accepted and ready to land.
REPOSITORY
R289 KNotifications
BRANCH
master
REVISION DETAIL
https://phabricator.kde.org/D6963
To: davidedmundson, mart
Cc: alexeymin, marten, vandenoever, mck182, dvratil, #frameworks
davidedmundson updated this revision to Diff 18912.
davidedmundson added a comment.
Remove startFdo flag
Keep windows away from DBus activation
Better handle the service owner changing on a potentially
DBus activated notification handler
REPOSITORY
R289 KNotifications
CHANGES
vandenoever added a comment.
Removing this call is in line with the advice in the dbus spec.
An improvement to the patch would be to remove `bool startfdo`.
Q_WS_WIN seems to suggest that windows always needs to start the service
manually. That case would break with this patch.
davidedmundson added a comment.
No, his one still blocks. This doesn't.
REPOSITORY
R289 KNotifications
REVISION DETAIL
https://phabricator.kde.org/D6963
To: davidedmundson
Cc: mck182, dvratil, #frameworks
davidedmundson created this revision.
Restricted Application added a project: Frameworks.
Restricted Application added a subscriber: Frameworks.
REVISION SUMMARY
We don't need to manually start the DBus service.
It blocks the calling app, and dbusServiceExists means that we will
always end