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

qwinrtcursor: Use QPointF as much as possible in pos/setPos

Browse Source 
Using QPoint for calculations can lead to rounding errors which can
result in a situation where
"QCursor::setPos(somePos); QCOMPARE(pos, somePos)" fails.

Change-Id: Iaba499f67abcf5ec66adfe93bfbd79ef0102e6d0
Reviewed-by: Friedemann Kleint <Friedemann.Kleint@qt.io>
This commit is contained in:
Oliver Wolff 2018-09-13 15:18:04 +02:00
parent fdcbf0b2a2
commit 7b00eab593
1 changed files with 7 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

13
src/plugins/platforms/winrt/qwinrtcursor.cpp
View File

@ -179,14 +179,15 @@ QPoint QWinRTCursor::pos() const
return hr;
});
Q_ASSERT_SUCCEEDED(hr);
QPoint position = QPoint(int(point.X), int(point.Y));
QPointF position(qreal(point.X), qreal(point.Y));
// If no cursor get_PointerPosition returns SHRT_MIN for x and y
if ((position.x() == SHRT_MIN && position.y() == SHRT_MIN) || FAILED(hr))
if ((int(position.x()) == SHRT_MIN && int(position.y()) == SHRT_MIN)
|| FAILED(hr))
return QPointF(Q_INFINITY, Q_INFINITY).toPoint();
position.rx() -= bounds.X;
position.ry() -= bounds.Y;
position.rx() -= qreal(bounds.X);
position.ry() -= qreal(bounds.Y);
position *= screen->scaleFactor();
return position;
return position.toPoint();
}
void QWinRTCursor::setPos(const QPoint &pos)
@ -195,7 +196,7 @@ void QWinRTCursor::setPos(const QPoint &pos)
Q_ASSERT(screen);
ComPtr<ICoreWindow> coreWindow = screen->coreWindow();
Q_ASSERT(coreWindow);
const QPointF scaledPos = pos / screen->scaleFactor();
const QPointF scaledPos = QPointF(pos) / screen->scaleFactor();
QWinRTScreen::MousePositionTransition t;
HRESULT hr = QEventDispatcherWinRT::runOnXamlThread([coreWindow, scaledPos, &t]() {
ComPtr<ICoreWindow2> coreWindow2;