bugfix(input): Double-clicking to place a building no longer performs double-click selection logic (#1608)

Stubbjax · web-flow · commit 1875dbfa296e · 2025-09-21T17:24:28.000+02:00
diff --git a/Generals/Code/GameEngine/Source/GameClient/MessageStream/PlaceEventTranslator.cpp b/Generals/Code/GameEngine/Source/GameClient/MessageStream/PlaceEventTranslator.cpp
@@ -62,6 +62,8 @@ GameMessageDisposition PlaceEventTranslator::translateGameMessage(const GameMess
 	{
 
 		//---------------------------------------------------------------------------------------------
+		// TheSuperHackers @bugfix Prevent double-clicks from falling through to other translators during building placement
+		case GameMessage::MSG_RAW_MOUSE_LEFT_DOUBLE_CLICK:
 		case GameMessage::MSG_RAW_MOUSE_LEFT_BUTTON_DOWN:
 		{
 			// if we're in a building placement mode, do the place and send to all players
diff --git a/GeneralsMD/Code/GameEngine/Source/GameClient/MessageStream/PlaceEventTranslator.cpp b/GeneralsMD/Code/GameEngine/Source/GameClient/MessageStream/PlaceEventTranslator.cpp
@@ -68,6 +68,8 @@ GameMessageDisposition PlaceEventTranslator::translateGameMessage(const GameMess
 	{
 
 		//---------------------------------------------------------------------------------------------
+		// TheSuperHackers @bugfix Prevent double-clicks from falling through to other translators during building placement
+		case GameMessage::MSG_RAW_MOUSE_LEFT_DOUBLE_CLICK:
 		case GameMessage::MSG_RAW_MOUSE_LEFT_BUTTON_DOWN:
 		{
 			// if we're in a building placement mode, do the place and send to all players

Original file line number	Diff line number	Diff line change
`@@ -62,6 +62,8 @@ GameMessageDisposition PlaceEventTranslator::translateGameMessage(const GameMess`
`62`	`62`	`{`
`63`	`63`
`64`	`64`	`//---------------------------------------------------------------------------------------------`
	`65`	`+ // TheSuperHackers @bugfix Prevent double-clicks from falling through to other translators during building placement`
	`66`	`+ case GameMessage::MSG_RAW_MOUSE_LEFT_DOUBLE_CLICK:`
`65`	`67`	`case GameMessage::MSG_RAW_MOUSE_LEFT_BUTTON_DOWN:`
`66`	`68`	`{`
`67`	`69`	`// if we're in a building placement mode, do the place and send to all players`
Original file line number	Diff line number	Diff line change
`@@ -68,6 +68,8 @@ GameMessageDisposition PlaceEventTranslator::translateGameMessage(const GameMess`
`68`	`68`	`{`
`69`	`69`
`70`	`70`	`//---------------------------------------------------------------------------------------------`
	`71`	`+ // TheSuperHackers @bugfix Prevent double-clicks from falling through to other translators during building placement`
	`72`	`+ case GameMessage::MSG_RAW_MOUSE_LEFT_DOUBLE_CLICK:`
`71`	`73`	`case GameMessage::MSG_RAW_MOUSE_LEFT_BUTTON_DOWN:`
`72`	`74`	`{`
`73`	`75`	`// if we're in a building placement mode, do the place and send to all players`