From 4814c4686f20d39accc47e8c44c057a80d476599 Mon Sep 17 00:00:00 2001 From: Vasily Date: Thu, 21 Feb 2019 16:11:14 +0100 Subject: [PATCH] Add brackets to optimize statement Not having brackets forces the compiler to see the statement as (xdelta * 360.0) * 8.0, which requires two `mulsd` operations. Adding the brackets only requires one. More details here https://github.com/OpenVR-Advanced-Settings/OpenVR-AdvancedSettings/pull/87 --- src/overlaycontroller.cpp | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/src/overlaycontroller.cpp b/src/overlaycontroller.cpp index 42c3a21..2b525b3 100644 --- a/src/overlaycontroller.cpp +++ b/src/overlaycontroller.cpp @@ -644,10 +644,10 @@ void OverlayController::mainEventLoop() m_ptLastMouse, m_pWindow->mapToGlobal( m_ptLastMouse ), QPoint(), - QPoint( static_cast( vrEvent.data.scroll.xdelta * 360.0f - * 8.0f ), - static_cast( vrEvent.data.scroll.ydelta * 360.0f - * 8.0f ) ), + QPoint( static_cast( vrEvent.data.scroll.xdelta + * ( 360.0f * 8.0f ) ), + static_cast( vrEvent.data.scroll.ydelta + * ( 360.0f * 8.0f ) ) ), 0, Qt::Vertical, m_lastMouseButtons,