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

libinput: Reorganize touch frame handler

Browse Source 
Avoid showing unnecessary "TouchFrame without registered device"
warnings. That should be reserved only for not having a device ready.
The touch point list's emptyiness is a different story - there we
should stop silently as that is not an error.

Change-Id: Icdb8b352351b70a7e1af2d3a1de3001dfb751aae
Reviewed-by: Shawn Rutledge <shawn.rutledge@digia.com>
This commit is contained in:
Laszlo Agocs 2015-03-25 10:21:12 +01:00
parent 0809a922a0
commit b1a9787f01
1 changed files with 15 additions and 11 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
src/platformsupport/input/libinput/qlibinputtouch.cpp
View File

@ -147,9 +147,16 @@ void QLibInputTouch::processTouchCancel(libinput_event_touch *e)
void QLibInputTouch::processTouchFrame(libinput_event_touch *e)
{
DeviceState *state = deviceState(e);
if (state->m_touchDevice && !state->m_points.isEmpty()) {
if (!state->m_touchDevice) {
qWarning("TouchFrame without registered device");
return;
}
if (state->m_points.isEmpty())
return;
QWindowSystemInterface::handleTouchEvent(Q_NULLPTR, state->m_touchDevice, state->m_points,
QGuiApplication::keyboardModifiers());
for (int i = 0; i < state->m_points.count(); ++i) {
QWindowSystemInterface::TouchPoint &tp(state->m_points[i]);
if (tp.state == Qt::TouchPointReleased)
@ -157,9 +164,6 @@ void QLibInputTouch::processTouchFrame(libinput_event_touch *e)
else if (tp.state == Qt::TouchPointPressed)
tp.state = Qt::TouchPointStationary;
}
} else {
qWarning("TouchFrame without registered device");
}
}
QT_END_NAMESPACE