Fixed issue where Unity adds object to selection on click + modifier key

Author: Sander van Rossen

RealtimeCSG/Assets/Plugins/RealtimeCSG/Editor/Scripts/Control/Managers/RectangleSelectionManager.cs

@@ -112,6 +112,9 @@ static void InitReflectedData()
 		static bool		mouseDragged        = false;
 		static Vector2	clickMousePosition  = MathConstants.zeroVector2;
 
+
+		static List<UnityEngine.Object> sFoundObjects = new List<UnityEngine.Object>();
+
 		// Update rectangle selection using reflection
 		// This is hacky & dangerous 
 		// LOOK AWAY NOW!
@@ -307,18 +310,32 @@ internal static void Update(SceneView sceneView)
 				case EventType.MouseDown:
 				{
 					rectClickDown = (Event.current.button == 0 && hotControl == s_RectSelectionID_instance);
+					var selectedObjects = Selection.objects;
+					sFoundObjects.Clear();
+					foreach (var obj in selectedObjects)
+					{
+						var component = obj as Component;
+						var gameObject = obj as GameObject;
+						var transform = obj as Transform;
+						if (!(component && component.GetComponent<GeneratedMeshes>()) &&
+							!(gameObject && gameObject.GetComponent<GeneratedMeshes>()) &&
+							!(transform && transform.GetComponent<Transform>()))
+							sFoundObjects.Add(obj);
+					}
 					clickMousePosition = Event.current.mousePosition;
 					mouseDragged = false;
 					break;
 				}
 				case EventType.MouseUp:
 				{
 					rectClickDown = false;
+					sFoundObjects.Clear();
 					break;
 				}
 				case EventType.MouseMove:
 				{
 					rectClickDown = false;
+					sFoundObjects.Clear();
 					break;
 				}
 				case EventType.Used:
@@ -340,20 +357,9 @@ internal static void Update(SceneView sceneView)
 							if (Selection.gameObjects != null)
 							{
 								var selectedObjects = Selection.objects;
-								var foundObjects = new List<UnityEngine.Object>();
-								foreach (var obj in selectedObjects)
-								{
-									var component = obj as Component;
-									var gameObject = obj as GameObject;
-									var transform = obj as Transform;
-									if (!(component && component.GetComponent<GeneratedMeshes>()) &&
-										!(gameObject && gameObject.GetComponent<GeneratedMeshes>()) &&
-										!(transform && transform.GetComponent<Transform>()))
-										foundObjects.Add(obj);
-								}
-								if (foundObjects.Count != selectedObjects.Length)
+								if (sFoundObjects.Count != selectedObjects.Length)
 								{
-									Selection.objects = foundObjects.ToArray();
+									Selection.objects = sFoundObjects.ToArray();
 								}
 							}
 
@@ -363,6 +369,7 @@ internal static void Update(SceneView sceneView)
 
 					}
 					rectClickDown = false;
+					sFoundObjects.Clear();
 					break;
 				}