Basic multiple windows working

GlitchyEngine/src/Application.bf

@@ -15,6 +15,9 @@ namespace GlitchyEngine
 		static Application s_Instance = null;
 
 		private Window _mainWindow;
+
+		private append List<Window> _windows = .();
+
 		private RendererAPI _rendererApi;
 
 		private bool _running = true;
@@ -53,6 +56,8 @@ namespace GlitchyEngine
 
 		public Settings Settings {get; private set;} = new .() ~ delete _;
 
+		public List<Window> Windows => _windows;
+
 		public this()
 		{
 			Profiler.ProfileFunction!();
@@ -122,7 +127,11 @@ namespace GlitchyEngine
 			delete _layerStack;
 
 			delete _rendererApi;
-			delete _mainWindow;
+
+			while (!_windows.IsEmpty)
+			{
+				delete _windows.Back;
+			}
 			
 			delete _gameTime;
 
@@ -197,8 +206,11 @@ namespace GlitchyEngine
 #if IMGUI
 					_imGuiLayer.ImGuiRender();
 #endif
-	
-					_mainWindow.SwapChain.Present();
+
+					for (Window window in _windows)
+					{
+						window.SwapChain.Present();
+					}
 				}
 			}
 		}

GlitchyEngine/src/Events/MouseEvent.bf

@@ -4,6 +4,7 @@ namespace GlitchyEngine.Events
 {
 	public class MouseMovedEvent : Event, IEvent
 	{
+		private Window _window;
 		private int32 _mouseX, _mouseY;
 
 		public override EventType EventType => .MouseMoved;
@@ -14,11 +15,14 @@ namespace GlitchyEngine.Events
 
 		public static EventType StaticType => .MouseMoved;
 
+		public Window Window => _window;
+
 		public int32 PositionX => _mouseX;
 		public int32 PositionY => _mouseY;
 
-		public this(int32 x, int32 y)
+		public this(Window window, int32 x, int32 y)
 		{
+			_window = window;
 			_mouseX = x;
 			_mouseY = y;
 		}
@@ -95,14 +99,18 @@ namespace GlitchyEngine.Events
 
 	public abstract class MouseButtonEvent : Event
 	{
+		protected Window _window;
 		protected MouseButton _mouseButton;
 
 		public override EventCategory Category => .Input | .Mouse;
 
+		public Window Window => _window;
+
 		public MouseButton MouseButton => _mouseButton;
 
-		protected this(MouseButton mouseButton)
+		protected this(Window window, MouseButton mouseButton)
 		{
+			_window = window;
 			_mouseButton = mouseButton;
 		}
 	}
@@ -115,7 +123,7 @@ namespace GlitchyEngine.Events
 
 		public static EventType StaticType => .MouseButtonPressed;
 
-		public this(MouseButton mouseButton) : base(mouseButton) {  }
+		public this(Window window, MouseButton mouseButton) : base(window, mouseButton) {  }
 
 		public override void ToString(String strBuffer)
 		{
@@ -132,7 +140,7 @@ namespace GlitchyEngine.Events
 
 		public static EventType StaticType => .MouseButtonReleased;
 
-		public this(MouseButton mouseButton) : base(mouseButton) {  }
+		public this(Window window, MouseButton mouseButton) : base(window, mouseButton) {  }
 
 		public override void ToString(String strBuffer)
 		{

GlitchyEngine/src/Platform/Windows/WindowsWindow.bf

@@ -162,6 +162,8 @@ namespace GlitchyEngine
 			_minMaxInfo.MaximumTrackingSize.x = int32.MaxValue;
 			_minMaxInfo.MaximumTrackingSize.y = int32.MaxValue;
 
+			Application.Instance.Windows.Add(this);
+
 			Init(desc);
 		}
 
@@ -181,7 +183,10 @@ namespace GlitchyEngine
 				Log.EngineLogger.Error($"Failed to destroy window: Message ({(int)res}): {res}");
 			}
 
+			// TODO manage window classes somehow
 			UnregisterClass(WindowClassName.ToScopedNativeWChar!(), _instanceHandle);
+
+			Application.Instance.Windows.Remove(this);
 		}
 
 		[LinkName(.C)]
@@ -217,7 +222,7 @@ namespace GlitchyEngine
 			{
 				uint32 lastError = GetLastError();
 				Log.EngineLogger.Error($"Failed to register window class. Message({(int)lastError}): {(HResult)lastError}");
-				Runtime.FatalError("Failed to register window class");
+				//Runtime.FatalError("Failed to register window class");
 			}
 
 			{
@@ -381,34 +386,34 @@ namespace GlitchyEngine
 				// Left mouse button
 			case WM_LBUTTONDOWN:
 				{
-					var event = scope MouseButtonPressedEvent(.LeftButton);
+					var event = scope MouseButtonPressedEvent(window, .LeftButton);
 					window._eventCallback(event);
 				}
 			case WM_LBUTTONUP:
 				{
-					var event = scope MouseButtonReleasedEvent(.LeftButton);
+					var event = scope MouseButtonReleasedEvent(window, .LeftButton);
 					window._eventCallback(event);
 				}
 				// Right mouse button
 			case WM_RBUTTONDOWN:
 				{
-					var event = scope MouseButtonPressedEvent(.RightButton);
+					var event = scope MouseButtonPressedEvent(window, .RightButton);
 					window._eventCallback(event);
 				}
 			case WM_RBUTTONUP:
 				{
-					var event = scope MouseButtonReleasedEvent(.RightButton);
+					var event = scope MouseButtonReleasedEvent(window, .RightButton);
 					window._eventCallback(event);
 				}
 				// Middle mouse button
 			case WM_MBUTTONDOWN:
 				{
-					var event = scope MouseButtonPressedEvent(.MiddleButton);
+					var event = scope MouseButtonPressedEvent(window, .MiddleButton);
 					window._eventCallback(event);
 				}
 			case WM_MBUTTONUP:
 				{
-					var event = scope MouseButtonReleasedEvent(.MiddleButton);
+					var event = scope MouseButtonReleasedEvent(window, .MiddleButton);
 					window._eventCallback(event);
 				}
 				// X button
@@ -420,7 +425,7 @@ namespace GlitchyEngine
 					else
 						button = .XButton2;
 
-					var event = scope MouseButtonPressedEvent(button);
+					var event = scope MouseButtonPressedEvent(window, button);
 					window._eventCallback(event);
 				}
 			case WM_XBUTTONUP:
@@ -431,7 +436,7 @@ namespace GlitchyEngine
 					else
 						button = .XButton2;
 
-					var event = scope MouseButtonReleasedEvent(button);
+					var event = scope MouseButtonReleasedEvent(window, button);
 					window._eventCallback(event);
 				}
 				// Vertical scrolling
@@ -453,8 +458,8 @@ namespace GlitchyEngine
 			case WM_MOUSEMOVE:
 				{
 					SplitHighAndLowOrder!(lParam, let x, let y);
-
-					var event = scope MouseMovedEvent(x, y);
+					
+					var event = scope MouseMovedEvent(window, x, y);
 					window._eventCallback(event);
 				}
 			case WM_INPUT:
@@ -470,8 +475,8 @@ namespace GlitchyEngine
 							RAWINPUT* raw = (.)rawData.CArray();
 							if (raw.Header.Type == RIM_TYPEMOUSE)
 							{
-								var event = scope MouseMovedEvent(raw.Data.Mouse.lLastX, raw.Data.Mouse.lLastY);
-								window._eventCallback(event);
+								var event = scope MouseMovedEvent(window, raw.Data.Mouse.lLastX, raw.Data.Mouse.lLastY);
+								//window._eventCallback(event);
 
 								// We accumulate the raw movements an collect them once each frame in WindowsInput.Impl_NewFrame()
 								window._rawMouseMovementAccumulator += int2(raw.Data.Mouse.lLastX, raw.Data.Mouse.lLastY);

GlitchyEngine/src/Window.bf

@@ -31,7 +31,7 @@ namespace GlitchyEngine
 
 		public delegate void EventCallback(Event e);
 
-		protected EventCallback _eventCallback ~ delete _;
+		protected EventCallback _eventCallback = new => DefaultEventHandler ~ delete _;
 
 		public SwapChain SwapChain => _swapChain;
 
@@ -105,7 +105,11 @@ namespace GlitchyEngine
 		public EventCallback EventCallback
 		{
 			get => _eventCallback;
-			set => _eventCallback = value;
+			set
+			{
+				delete _eventCallback;
+				_eventCallback = value;
+			}
 		}
 
 		public extern void Update();
@@ -114,5 +118,10 @@ namespace GlitchyEngine
 		 * Sets the Icon of the window to the given file.
 		 */
 		public extern Result<void> SetIcon(StringView filePath);
+
+		private void DefaultEventHandler(Event e)
+		{
+			Log.EngineLogger.Error($"No event handler registered for window.");
+		}
 	}
 }