https://bugs.kde.org/show_bug.cgi?id=382444

David Edmundson <k...@davidedmundson.co.uk> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
         Resolution|---                         |FIXED
      Latest Commit|                            |https://commits.kde.org/kno
                   |                            |tifications/1c97e1d9741fd15
                   |                            |962474f47932dd09728dae76b
             Status|CONFIRMED                   |RESOLVED

--- Comment #9 from David Edmundson <k...@davidedmundson.co.uk> ---
Git commit 1c97e1d9741fd15962474f47932dd09728dae76b by David Edmundson.
Committed on 29/08/2017 at 08:44.
Pushed by davidedmundson into branch 'master'.

Don't block starting notification service

Summary:
We don't need to manually start the DBus service.
It blocks the calling app, and dbusServiceExists means that we will
always end up going the DBus route over a popup anyway, so it won't
do anything useful.

The service (in the plasma case plasma-wait-for-name) will be started
automatically when we actually send the notification.

Also fix d-dbusServiceExists being cleared to false when the first
service owner exits.

Subscribers: #frameworks

Tags: #frameworks

Differential Revision: https://phabricator.kde.org/D6963

M  +11   -17   src/notifybypopup.cpp

https://commits.kde.org/knotifications/1c97e1d9741fd15962474f47932dd09728dae76b

-- 
You are receiving this mail because:
You are watching all bug changes.

Reply via email to