From 03a691cecb91ea91394d40b0b7e1b15210fc5687 Mon Sep 17 00:00:00 2001 From: Shawn Rutledge Date: Tue, 12 Jul 2022 15:54:40 +0200 Subject: [PATCH] Change "delivering touch to same window" warnings to debug messages When you touch-tap a QPushButton, and that opens a popup, it's normal that the touch release is delivered to the main window rather than the popup: in fact QWidgetWindow::handleMouseEvent() expects to see a synth-mouse release. No warning is needed. Amends efc02f9cc301f98c77079adae026ffd07f50d5ab There may be other cases when it should have remained a warning (it's been there since 2692237bb1b0c0f50b7cc5d920eb8ab065063d47), but that's hypothetical at this point. Pick-to: 6.2 6.3 6.4 Task-number: QTBUG-103706 Change-Id: Ie8cc62af4d173ada5138fd099d9f02bd41f4cff4 Reviewed-by: Volker Hilsheimer --- src/gui/kernel/qguiapplication.cpp | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/src/gui/kernel/qguiapplication.cpp b/src/gui/kernel/qguiapplication.cpp index 6f89efb26c..efa836522a 100644 --- a/src/gui/kernel/qguiapplication.cpp +++ b/src/gui/kernel/qguiapplication.cpp @@ -2893,15 +2893,15 @@ void QGuiApplicationPrivate::processTouchEvent(QWindowSystemInterfacePrivate::To case QEventPoint::State::Released: if (Q_UNLIKELY(!window.isNull() && window != QMutableEventPoint::window(ep))) - qCWarning(lcPtrDispatch) << "delivering touch release to same window" - << QMutableEventPoint::window(ep) << "not" << window.data(); + qCDebug(lcPtrDispatch) << "delivering touch release to same window" + << QMutableEventPoint::window(ep) << "not" << window.data(); window = QMutableEventPoint::window(ep); break; default: // update or stationary if (Q_UNLIKELY(!window.isNull() && window != QMutableEventPoint::window(ep))) - qCWarning(lcPtrDispatch) << "delivering touch update to same window" - << QMutableEventPoint::window(ep) << "not" << window.data(); + qCDebug(lcPtrDispatch) << "delivering touch update to same window" + << QMutableEventPoint::window(ep) << "not" << window.data(); window = QMutableEventPoint::window(ep); break; }