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

tablet: small cleanups

Browse source 
Add missing &
Remove unnecessary namespace
Consistant m_ prefix
This commit is contained in:
Aleix Pol 2020-12-16 02:44:17 +01:00
parent 03fc51ef2f
commit 9641aa42ec
1 changed files with 12 additions and 10 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

22
src/wayland/tablet_v2_interface.cpp
View file

@ -20,7 +20,7 @@ static int s_version = 1;
class TabletV2InterfacePrivate : public QtWaylandServer::zwp_tablet_v2 class TabletV2InterfacePrivate : public QtWaylandServer::zwp_tablet_v2
{ {
public: public:
TabletV2InterfacePrivate(TabletV2Interface *q, uint32_t vendorId, uint32_t productId, const QString name, const QStringList &paths) TabletV2InterfacePrivate(TabletV2Interface *q, uint32_t vendorId, uint32_t productId, const QString &name, const QStringList &paths)
: zwp_tablet_v2() : zwp_tablet_v2()
, q(q) , q(q)
, m_vendorId(vendorId) , m_vendorId(vendorId)
@ -33,14 +33,14 @@ public:
wl_resource *resourceForSurface(SurfaceInterface *surface) const wl_resource *resourceForSurface(SurfaceInterface *surface) const
{ {
ClientConnection *client = surface->client(); ClientConnection *client = surface->client();
QtWaylandServer::zwp_tablet_v2::Resource *r = resourceMap().value(*client); Resource *r = resourceMap().value(*client);
return r ? r->handle : nullptr; return r ? r->handle : nullptr;
} }
void zwp_tablet_v2_destroy_resource(QtWaylandServer::zwp_tablet_v2::Resource * resource) override void zwp_tablet_v2_destroy_resource(QtWaylandServer::zwp_tablet_v2::Resource * resource) override
{ {
Q_UNUSED(resource); Q_UNUSED(resource);
if (removed && resourceMap().isEmpty()) { if (m_removed && resourceMap().isEmpty()) {
delete q; delete q;
} }
} }
@ -50,7 +50,7 @@ public:
const uint32_t m_productId; const uint32_t m_productId;
const QString m_name; const QString m_name;
const QStringList m_paths; const QStringList m_paths;
bool removed = false; bool m_removed = false;
}; };
TabletV2Interface::TabletV2Interface(uint32_t vendorId, uint32_t productId, TabletV2Interface::TabletV2Interface(uint32_t vendorId, uint32_t productId,
@ -70,7 +70,7 @@ bool TabletV2Interface::isSurfaceSupported(SurfaceInterface *surface) const
void TabletV2Interface::sendRemoved() void TabletV2Interface::sendRemoved()
{ {
d->removed = true; d->m_removed = true;
for (QtWaylandServer::zwp_tablet_v2::Resource *resource : d->resourceMap()) { for (QtWaylandServer::zwp_tablet_v2::Resource *resource : d->resourceMap()) {
d->send_removed(resource->handle); d->send_removed(resource->handle);
} }
@ -83,12 +83,14 @@ public:
void update(quint32 serial, SurfaceInterface *surface, const QPoint &hotspot) void update(quint32 serial, SurfaceInterface *surface, const QPoint &hotspot)
{ {
const bool diff = m_serial != serial && m_surface != surface && m_hotspot != hotspot; const bool diff = m_serial != serial || m_surface != surface || m_hotspot != hotspot;
m_serial = serial; if (diff) {
m_surface = surface; m_serial = serial;
m_hotspot = hotspot; m_surface = surface;
if (diff) m_hotspot = hotspot;
Q_EMIT q->changed(); Q_EMIT q->changed();
}
} }
TabletCursorV2 *const q; TabletCursorV2 *const q;