Fix middle mouse button click detection to use real-time instead of frames

Author: bobtista

Generals/Code/GameEngine/Include/GameClient/LookAtXlat.h

@@ -75,7 +75,7 @@ class LookAtTranslator : public GameMessageTranslator
 	Bool m_isRotating;					// set to true if we are in the act of MMB rotating
 	Bool m_isPitching;					// set to true if we are in the act of ALT pitch rotation
 	Bool m_isChangingFOV;			// set to true if we are in the act of changing the field of view
-	UnsignedInt m_timestamp;				// set when button goes down
+	UnsignedInt m_middleButtonDownTimeMS;				// real-time in milliseconds when middle button goes down
 	DrawableID m_lastPlaneID;
 	ViewLocation m_viewLocation[ MAX_VIEW_LOCS ];
 	ScrollType m_scrollType;

Generals/Code/GameEngine/Source/GameClient/MessageStream/LookAtXlat.cpp

@@ -281,8 +281,8 @@ GameMessageDisposition LookAtTranslator::translateGameMessage(const GameMessage
 			{
 				stopScrolling();
 			}
-			break;
-		}
+		break;
+	}
 
 		//-----------------------------------------------------------------------------
 		case GameMessage::MSG_RAW_MOUSE_MIDDLE_BUTTON_DOWN:
@@ -293,7 +293,7 @@ GameMessageDisposition LookAtTranslator::translateGameMessage(const GameMessage
 			m_anchor = msg->getArgument( 0 )->pixel;
 			m_originalAnchor = msg->getArgument( 0 )->pixel;
 			m_currentPos = msg->getArgument( 0 )->pixel;
-			m_timestamp = TheGameClient->getFrame();
+			m_middleButtonDownTimeMS = timeGetTime();
 			break;
 		}
 
@@ -302,7 +302,7 @@ GameMessageDisposition LookAtTranslator::translateGameMessage(const GameMessage
 		{
 			m_lastMouseMoveFrame = TheGameLogic->getFrame();
 
-			const UnsignedInt CLICK_DURATION = 5;
+			const UnsignedInt CLICK_DURATION_MS = 167;
 			const UnsignedInt PIXEL_OFFSET = 5;
 
 			m_isRotating = false;
@@ -311,7 +311,7 @@ GameMessageDisposition LookAtTranslator::translateGameMessage(const GameMessage
 			Int dy = m_currentPos.y-m_originalAnchor.y;
 			Bool didMove = dx>PIXEL_OFFSET || dy>PIXEL_OFFSET;
 			// if middle button is "clicked", reset to "home" orientation
-			if (!didMove && TheGameClient->getFrame() - m_timestamp < CLICK_DURATION)
+			if (!didMove && timeGetTime() - m_middleButtonDownTimeMS < CLICK_DURATION_MS)
 			{
 				TheTacticalView->setAngleAndPitchToDefault();
 				TheTacticalView->setZoomToDefault();

GeneralsMD/Code/GameEngine/Include/GameClient/LookAtXlat.h

@@ -75,7 +75,7 @@ class LookAtTranslator : public GameMessageTranslator
 	Bool m_isRotating;					// set to true if we are in the act of MMB rotating
 	Bool m_isPitching;					// set to true if we are in the act of ALT pitch rotation
 	Bool m_isChangingFOV;			// set to true if we are in the act of changing the field of view
-	UnsignedInt m_timestamp;				// set when button goes down
+	UnsignedInt m_middleButtonDownTimeMS;				// real-time in milliseconds when middle button goes down
 	DrawableID m_lastPlaneID;
 	ViewLocation m_viewLocation[ MAX_VIEW_LOCS ];
 	ScrollType m_scrollType;

GeneralsMD/Code/GameEngine/Source/GameClient/MessageStream/LookAtXlat.cpp

@@ -280,8 +280,8 @@ GameMessageDisposition LookAtTranslator::translateGameMessage(const GameMessage
 			{
 				stopScrolling();
 			}
-			break;
-		}
+		break;
+	}
 
 		//-----------------------------------------------------------------------------
 		case GameMessage::MSG_RAW_MOUSE_MIDDLE_BUTTON_DOWN:
@@ -292,7 +292,7 @@ GameMessageDisposition LookAtTranslator::translateGameMessage(const GameMessage
 			m_anchor = msg->getArgument( 0 )->pixel;
 			m_originalAnchor = msg->getArgument( 0 )->pixel;
 			m_currentPos = msg->getArgument( 0 )->pixel;
-			m_timestamp = TheGameClient->getFrame();
+			m_middleButtonDownTimeMS = timeGetTime();
 			break;
 		}
 
@@ -301,7 +301,7 @@ GameMessageDisposition LookAtTranslator::translateGameMessage(const GameMessage
 		{
 			m_lastMouseMoveFrame = TheGameLogic->getFrame();
 
-			const UnsignedInt CLICK_DURATION = 5;
+			const UnsignedInt CLICK_DURATION_MS = 167;
 			const UnsignedInt PIXEL_OFFSET = 5;
 
 			m_isRotating = false;
@@ -310,7 +310,7 @@ GameMessageDisposition LookAtTranslator::translateGameMessage(const GameMessage
 			Int dy = m_currentPos.y-m_originalAnchor.y;
 			Bool didMove = dx>PIXEL_OFFSET || dy>PIXEL_OFFSET;
 			// if middle button is "clicked", reset to "home" orientation
-			if (!didMove && TheGameClient->getFrame() - m_timestamp < CLICK_DURATION)
+			if (!didMove && timeGetTime() - m_middleButtonDownTimeMS < CLICK_DURATION_MS)
 			{
 				TheTacticalView->setAngleAndPitchToDefault();
 				TheTacticalView->setZoomToDefault();