https://bugs.kde.org/show_bug.cgi?id=374665
David Edmundson <k...@davidedmundson.co.uk> changed:
What |Removed |Added
----------------------------------------------------------------------------
Resolution|--- |FIXED
Latest Commit| |https://commits.kde.org/kwa
| |yland-integration/34fc30c4f
| |7ca6d461ebfa0b7691e05edf2b0
| |e40e
Status|UNCONFIRMED |RESOLVED
--- Comment #12 from David Edmundson <k...@davidedmundson.co.uk> ---
Git commit 34fc30c4f7ca6d461ebfa0b7691e05edf2b0e40e by David Edmundson.
Committed on 05/05/2017 at 22:28.
Pushed by davidedmundson into branch 'master'.
Cleanup wayland resources used in kwindowsystem before they're deleted by QPA
Summary:
When the Qt QPA closes it deletes the wl_display
deleting wl_display deletes the wl_registry
when we destroy the kwayland wrapper we double delete
as the windowsystem integration is a singleton we're not deleted till
after qApp.
By manually releasing the wayland parts early we avoid that problem,
whilst still being safe if any app did make a KWindowSystem call during
shutdown.
Test Plan:
Locked screen, no longer crashed when typing in password
Opened system settings, created a blurred tooltip then closed. No longer
crashed
Making a unit test that runs during qApp shutdown will be incredibly difficult.
Reviewers: #plasma
Subscribers: plasma-devel
Tags: #plasma
Differential Revision: https://phabricator.kde.org/D5723
M +30 -0 src/windowsystem/waylandintegration.cpp
https://commits.kde.org/kwayland-integration/34fc30c4f7ca6d461ebfa0b7691e05edf2b0e40e
--
You are receiving this mail because:
You are watching all bug changes.
