Skip to content

Add missing QPointer

Vlad Zahorodnii requested to merge work/zzag/effect-screen-qpointer into master

The Compositor is destroyed after the Workspace, so the placeholder has shorter lifespan. We need to guard the destructor of EffectScreenImpl from accessing a dangling pointer. For what it's worth, the destructor of EffectScreenImpl was written with QPointer in mind, but it was never added.

Merge request reports