Merge lp:~gerboland/unity-2d/keymonitor-shutdown-fixes into lp:unity-2d
Proposed by
Gerry Boland
Status: | Merged |
---|---|
Approved by: | Olivier Tilloy |
Approved revision: | 770 |
Merged at revision: | 792 |
Proposed branch: | lp:~gerboland/unity-2d/keymonitor-shutdown-fixes |
Merge into: | lp:unity-2d |
Diff against target: |
108 lines (+22/-11) 2 files modified
libunity-2d-private/src/keymonitor.cpp (+17/-7) libunity-2d-private/src/keymonitor.h (+5/-4) |
To merge this branch: | bzr merge lp:~gerboland/unity-2d/keymonitor-shutdown-fixes |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Olivier Tilloy (community) | Approve | ||
Michał Sawicz | Pending | ||
Review via email: mp+80722@code.launchpad.net |
Description of the change
Rewrite KeyMonitor XEvent handler to use QSocketNotifier instead of separate thread. This singleton will now be deconstructed properly on application quit. Also free X11 objects when they're unnecessary.
To post a comment you must log in.
I tested this, but I don't see an improvement on session logout time.