https://bugs.kde.org/show_bug.cgi?id=494003
Akseli Lahtinen <akse...@akselmo.dev> changed:
What |Removed |Added
----------------------------------------------------------------------------
Latest Commit| |https://invent.kde.org/plas
| |ma/plasma-workspace/-/commi
| |t/20255a4c54891c349d322a2e9
| |92c136e041b9c22
Resolution|--- |FIXED
Status|ASSIGNED |RESOLVED
--- Comment #6 from Akseli Lahtinen <akse...@akselmo.dev> ---
Git commit 20255a4c54891c349d322a2e992c136e041b9c22 by Akseli Lahtinen.
Committed on 04/10/2024 at 11:17.
Pushed by akselmo into branch 'master'.
systemtray.cpp: Make AppletRemoved signal a singleshot connection
If user disables an entry in system settings, this connection
will end up in an endless loop:
When the `cleanupPlugin` is called,
we eventually end up calling
`SystemTray::onEnabledAppletsChanged`
through `SystemTraySettings::enabledPluginsChanged` signal.
There we delete applet and applets deconstructor calls
`appletDeleted`.
This then ends up in `ContainmentPrivate::appletDeleted` where
the signal `appletRemoved` is emitted.
This then makes the signal run over this same connection, then
`cleanupPlugin`.. And it loops.
Adding `Qt::SingleShotConnection` disconnects the connection
after it's called, so it won't get stuck in a loop.
M +15 -10 applets/systemtray/systemtray.cpp
https://invent.kde.org/plasma/plasma-workspace/-/commit/20255a4c54891c349d322a2e992c136e041b9c22
--
You are receiving this mail because:
You are watching all bug changes.
