1berry/qtbase
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

QWaylandShellIntegration: pin vtable to a single TU

Browse Source 
Fixes -Wweak-vtables, at least on the base class; the implementation
is a template, so can't be fixed.

Pick-to: 6.9
Change-Id: Ic697614995042e3e0cf130bbaf1bc21a891b497b
Reviewed-by: Volker Hilsheimer <volker.hilsheimer@qt.io>
This commit is contained in:
Marc Mutz 2025-01-05 12:05:38 +01:00
parent 9bb48c315c
commit 8fb0ceeb95
2 changed files with 4 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
src/plugins/platforms/wayland/shellintegration/qwaylandshellintegration.cpp
View File

@ -8,6 +8,9 @@ QT_BEGIN_NAMESPACE
namespace QtWaylandClient {
QWaylandShellIntegration::~QWaylandShellIntegration()
= default; // MUST stay empty until Qt 7 (was inline in Qt < 6.9)
wl_surface *QWaylandShellIntegration::wlSurfaceForWindow(QWaylandWindow *window)
{
return window->wlSurface();

2
src/plugins/platforms/wayland/shellintegration/qwaylandshellintegration_p.h
View File

@ -40,7 +40,7 @@ class Q_WAYLANDCLIENT_EXPORT QWaylandShellIntegration
{
public:
QWaylandShellIntegration() {}
virtual ~QWaylandShellIntegration() {}
virtual ~QWaylandShellIntegration();
virtual bool initialize(QWaylandDisplay *display) = 0;
virtual QWaylandShellSurface *createShellSurface(QWaylandWindow *window) = 0;