yukijoou/kwin
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Fix ownership of SlideInterface

Browse source 
Similar to other interfaces, the lifetime of the SlideInterface should
be bound to the associated resource.
This commit is contained in:
Vlad Zahorodnii 2021-10-31 12:37:41 +02:00
parent 12946a9ec9
commit fb672f335a
2 changed files with 4 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

7
src/wayland/slide_interface.cpp
View file

@ -45,7 +45,7 @@ void SlideManagerInterfacePrivate::org_kde_kwin_slide_manager_create(Resource *r
return;
}
auto slide = new SlideInterface(q, slide_resource);
auto slide = new SlideInterface(slide_resource);
SurfaceInterfacePrivate *surfacePrivate = SurfaceInterfacePrivate::get(s);
surfacePrivate->setSlide(QPointer<SlideInterface>(slide));
}
@ -137,9 +137,8 @@ SlideInterfacePrivate::SlideInterfacePrivate(SlideInterface *_q, wl_resource *re
{
}
SlideInterface::SlideInterface(SlideManagerInterface *manager, wl_resource *resource)
: QObject(manager)
, d(new SlideInterfacePrivate(this, resource))
SlideInterface::SlideInterface(wl_resource *resource)
: d(new SlideInterfacePrivate(this, resource))
{
}

2
src/wayland/slide_interface.h
View file

@ -56,7 +56,7 @@ public:
qint32 offset() const;
private:
explicit SlideInterface(SlideManagerInterface *manager, wl_resource *resource);
explicit SlideInterface(wl_resource *resource);
friend class SlideManagerInterfacePrivate;
QScopedPointer<SlideInterfacePrivate> d;