From c2372bc9a80c4515db70a272512c0bb4f49f1d2f Mon Sep 17 00:00:00 2001
From: "Jerry (Xinyu Hou)" <xinyu@synergy-project.org>
Date: Fri, 28 Oct 2016 05:24:07 -0700
Subject: [PATCH] #5186 Remove accumulate fraction DPI calculation

---
 src/lib/platform/MSWindowsScreen.cpp | 24 +++---------------------
 src/lib/platform/MSWindowsScreen.h   |  7 -------
 2 files changed, 3 insertions(+), 28 deletions(-)

diff --git a/src/lib/platform/MSWindowsScreen.cpp b/src/lib/platform/MSWindowsScreen.cpp
index 1ab3adf0..d6466b9b 100644
--- a/src/lib/platform/MSWindowsScreen.cpp
+++ b/src/lib/platform/MSWindowsScreen.cpp
@@ -104,7 +104,6 @@ MSWindowsScreen::MSWindowsScreen(
 	m_xCenter(0), m_yCenter(0),
 	m_multimon(false),
 	m_xCursor(0), m_yCursor(0),
-	m_xFractionalMove(0.0f), m_yFractionalMove(0.0f),
 	m_sequenceNumber(0),
 	m_mark(0),
 	m_markReceived(0),
@@ -570,21 +569,6 @@ void MSWindowsScreen::saveMousePosition(SInt32 x, SInt32 y) {
 	LOG((CLOG_DEBUG5 "saved mouse position for next delta: %+d,%+d", x,y));
 }
 
-void MSWindowsScreen::accumulateFractionalMove(float x, float y, SInt32& intX, SInt32& intY)
-{
-	// Accumulate together the move into the running total
-	m_xFractionalMove += x;
-	m_yFractionalMove += y;
-
-	// Return the integer part
-	intX = (SInt32)m_xFractionalMove;
-	intY = (SInt32)m_yFractionalMove;
-
-	// And keep only the fractional part
-	m_xFractionalMove -= intX;
-	m_yFractionalMove -= intY;
-}
-
 UInt32
 MSWindowsScreen::registerHotKey(KeyID key, KeyModifierMask mask)
 {
@@ -1365,8 +1349,8 @@ MSWindowsScreen::onMouseMove(SInt32 mx, SInt32 my)
 {
 	// compute motion delta (relative to the last known
 	// mouse position)
-	float x = (float)mx - m_xCursor;
-	float y = (float)my - m_yCursor;
+	SInt32 x = mx - m_xCursor;
+	SInt32 y = my - m_yCursor;
 
 	LOG((CLOG_DEBUG3
 		"mouse move - motion delta: %+d=(%+d - %+d),%+d=(%+d - %+d)",
@@ -1416,9 +1400,7 @@ MSWindowsScreen::onMouseMove(SInt32 mx, SInt32 my)
 		}
 		else {
 			// send motion
-			SInt32 ix, iy;
-			accumulateFractionalMove(x, y, ix, iy);
-			sendEvent(m_events->forIPrimaryScreen().motionOnSecondary(), MotionInfo::alloc(ix, iy));
+			sendEvent(m_events->forIPrimaryScreen().motionOnSecondary(), MotionInfo::alloc(x, y));
 		}
 	}
 
diff --git a/src/lib/platform/MSWindowsScreen.h b/src/lib/platform/MSWindowsScreen.h
index 4d947be7..b1ae0f94 100644
--- a/src/lib/platform/MSWindowsScreen.h
+++ b/src/lib/platform/MSWindowsScreen.h
@@ -216,10 +216,6 @@ private: // HACK
 	// save last position of mouse to compute next delta movement
 	void saveMousePosition(SInt32 x, SInt32 y);
 
-	// accumulates together a series of fractional pixel moves, each time
-	// taking away and returning just the integer part of the running total.
-	void accumulateFractionalMove(float x, float y, SInt32& intX, SInt32& intY);
-
 	// check if it is a modifier key repeating message
 	bool				isModifierRepeat(KeyModifierMask oldState, 
 							KeyModifierMask state, WPARAM wParam) const;
@@ -270,9 +266,6 @@ private:
 	// last mouse position
 	SInt32				m_xCursor, m_yCursor;
 
-	// accumulated fractional pixel moves
-	float				m_xFractionalMove, m_yFractionalMove;
-
 	// last clipboard
 	UInt32				m_sequenceNumber;